Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-27358

Google maps key not working in UI & scrollWheel not boolean

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: High High
    • Resolution: Fixed
    • Affects Version/s: 1.9.0, 1.9.1-rc
    • Fix Version/s: QA tracked issues, 1.7.4-rc, 1.10.0
    • Labels:
    • Environment:

      Operating System: Debian 8
      PHP Version: 5.6.30-0+deb8u1
      Database and version: Mysql 5.5.54-0+deb8u1
      Browser (and version): Firefox 53
      Env: Prod

      Description

      While validating EZP-26068, when I tried to use a content type that has a ezgmaplocation field, I have the following error

      10:45:54.876 "Google Maps API error: MissingKeyMapError https://developers.google.com/maps/documentation/javascript/error-messages#missing-key-map-error" 1 js:38:315
      	_.rb https://maps.googleapis.com/maps/api/js:38:315
      	Tm/this.f</< https://maps.googleapis.com/maps-api-v3/api/js/28/18/intl/pt_ALL/common.js:51:3
      	_.Gw</< https://maps.googleapis.com/maps-api-v3/api/js/28/18/intl/pt_ALL/common.js:194:171
      	c https://maps.googleapis.com/maps-api-v3/api/js/28/18/intl/pt_ALL/common.js:45:402
      	<anonymous> https://maps.googleapis.com/maps/api/js/AuthenticationService.Authenticate:1:22
      

      Steps to reproduce:
      -Set a valid google_maps key in app/config/ezplatform.yml

      system/site group/api_keys: { google_maps: "<yourKey>" }
      

      -create content type that has a ezgmapslocation
      -open your developer tools
      -create a content of the previously created content type
      The error will appear on developer tools and we can't see the map

        Issue Links

          Activity

          Hide
          Paulo Nunes (Inactive) added a comment -

          i'l close this issue because i have it/not depending on the environment I am at....

          Show
          Paulo Nunes (Inactive) added a comment - i'l close this issue because i have it/not depending on the environment I am at....
          Hide
          André Rømcke added a comment - - edited

          Reopening as the problem here is that key is not set, it's broken after it was changed to be an array as reported on Slack. The random bit that confused both me and QA is that google maps sometime let you use it for a few times without API keys, hiding the issue.

          Show
          André Rømcke added a comment - - edited Reopening as the problem here is that key is not set, it's broken after it was changed to be an array as reported on Slack. The random bit that confused both me and QA is that google maps sometime let you use it for a few times without API keys, hiding the issue.
          Hide
          André Rømcke added a comment -

          Additional issue found in EZP-26068 is that the added scrollWheel parameter needs to be treated internally like a string, like draggable argument.

          Both issues covered in PR: https://github.com/ezsystems/ezpublish-kernel/pull/2005

          Show
          André Rømcke added a comment - Additional issue found in EZP-26068 is that the added scrollWheel parameter needs to be treated internally like a string, like draggable argument. Both issues covered in PR: https://github.com/ezsystems/ezpublish-kernel/pull/2005
          Hide
          Io Sol Inf added a comment -

          Thank you, André Rømcke. The parameter type change will break functionality once this PR is merged, but it is more coherent with the remaining parameters now. Well done.

          Show
          Io Sol Inf added a comment - Thank you, André Rømcke . The parameter type change will break functionality once this PR is merged, but it is more coherent with the remaining parameters now. Well done.
          Show
          André Rømcke added a comment - Merged: https://github.com/ezsystems/ezpublish-kernel/commit/568aa9991f996407f33e62e80149f94f08de2ebc
          Hide
          Paulo Nunes (Inactive) added a comment -

          QA Approved

          Show
          Paulo Nunes (Inactive) added a comment - QA Approved

            People

            • Assignee:
              Unassigned
              Reporter:
              Paulo Nunes (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: