Skip to content

Commit

Permalink
fix (#18264)
Browse files Browse the repository at this point in the history
  • Loading branch information
zhouyx committed Sep 21, 2018
1 parent eb5e741 commit d89a34f
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions extensions/amp-consent/0.1/amp-consent.js
Expand Up @@ -31,11 +31,12 @@ import {
getSourceUrl,
resolveRelativeUrl,
} from '../../../src/url';
import {deepMerge, dict, map} from '../../../src/utils/object';
import {deepMerge, dict, hasOwn, map} from '../../../src/utils/object';
import {dev, user} from '../../../src/log';
import {getChildJsonConfig} from '../../../src/json';
import {getData} from '../../../src/event-helper';
import {getServicePromiseForDoc} from '../../../src/service';

import {isEnumValue} from '../../../src/types';
import {isExperimentOn} from '../../../src/experiments';
import {toggle} from '../../../src/style';
Expand Down Expand Up @@ -384,7 +385,7 @@ export class AmpConsent extends AMP.BaseElement {
promptPromise =
this.getConsentRemote_(instanceId).then(remoteConfigResponse => {
if (!remoteConfigResponse ||
!remoteConfigResponse['promptIfUnknown']) {
!hasOwn(remoteConfigResponse, 'promptIfUnknown')) {
this.user().error(TAG, 'Expecting promptIfUnknown from ' +
'checkConsentHref when promptIfUnknownForGeoGroup is not ' +
'specified');
Expand Down

0 comments on commit d89a34f

Please sign in to comment.