Reproduction:

  1. Navigate to Structure > Openlayers > Layers.
  2. Clone the "Google Maps Satellite"
  3. Edit the "Layer Machine Name" to "google_satellite_zoom_level_22", "Layer Description" to "Google Maps Satellite Imagery, enhanced zoom." and "Number of Zoom Levels" to "22".
  4. Click "Save" to attempt to save the layer.

Get the following warning:

Strict warning: Only variables should be passed by reference in openlayers_layer_type->save() (line 950 of [...]\drupal\sites\all\modules\contrib\openlayers\openlayers.module).

and a confirmation message that the layer was saved.

The same warning is raised when we try to edit the layer we just created/cloned.

Comments

nod_’s picture

Version: 7.x-2.0-beta1 » 7.x-2.x-dev
Category: support » bug
GeertWil’s picture

Version: 7.x-2.x-dev » 7.x-3.x-dev

I have the same issue when trying to add a kml layer.

I created a kml file that works fine in Google Maps and Google Earth. When I try to add the layer, I get the same error. In my case Drupal says the error happened in line 951, but I am working with dev version 3.x and nog 2.x.

GeertWil’s picture

The error dissappears when I switch back to version 7.x-2.0-beta, but then my old error messages show up again. The real list of error messages is much longer, I included only the first two ones.

Strict warning: Declaration of openlayers_layer_type_cloudmade::options_form() should be compatible with that of openlayers_layer_type::options_form() in ctools_plugin_load_includes() (regel 468 van C:\xampp\htdocs\btm\sites\all\modules\ctools\includes\plugins.inc).
Strict warning: Declaration of openlayers_layer_type_geojson::options_form() should be compatible with that of openlayers_layer_type::options_form() in ctools_plugin_load_includes() (regel 468 van C:\xampp\htdocs\btm\sites\all\modules\ctools\includes\plugins.inc).

zzolo’s picture

Version: 7.x-3.x-dev » 7.x-2.x-dev
Category: bug » support
Status: Active » Postponed (maintainer needs more info)

Hey @George Petsagourakis, are you still getting this warning. I cannot reproduce.

Hey @GeertWil, please make sure that your issue is the same as the original, and if not, please open another issue. Also, there is no reason to use the 3.x version of this module at the moment.

petsagouris’s picture

@zzolo I've moved on since then. I can't get back there now, sorry.

zzolo’s picture

Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)

Thanks @George.

zzolo’s picture

Issue summary: View changes

Removed path data.