Fixes publisher banner being empty in the db

Resolves brave/brave-browser#5907
brave · Sep 5, 2019 · 8cd8bb9 · 8cd8bb9
1 parent ce57c1b
commit 8cd8bb9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/vendor/bat-native-ledger/include/bat/ledger/public/interfaces/ledger.mojom b/vendor/bat-native-ledger/include/bat/ledger/public/interfaces/ledger.mojom
@@ -237,5 +237,5 @@ struct ServerPublisherInfo {
  PublisherStatus status = PublisherStatus.NOT_VERIFIED;
  bool excluded = false;
  string address;
- PublisherBanner banner;
+ PublisherBanner? banner;
 };
diff --git a/vendor/bat-native-ledger/src/bat/ledger/internal/publisher/publisher.cc b/vendor/bat-native-ledger/src/bat/ledger/internal/publisher/publisher.cc
@@ -888,7 +888,10 @@ void Publisher::OnGetPublisherBanner(
  auto banner = ledger::PublisherBanner::New();
 
  if (info) {
- banner = info->banner->Clone();
+ if (info->banner) {
+ banner = info->banner->Clone();
+ }
+
  banner->status = info->status;
  }