when I go to Location > Map Links I ge this error

theme.inc on line 617.
AfghanistanNone supported.Aland IslandsNone supported.AlbaniaNone supported.AlgeriaNone supported.American SamoaNone supported.AndorraNone supported.AngolaNone supported.AnguillaNone supported.AntarcticaNone supported.Antigua and BarbudaNone supported.ArgentinaNone supported.ArmeniaNone supported.ArubaNone supported.Australia
Google Maps (Terms of Use)
Yahoo! Maps (Terms of Use)
MapQuest (Terms of Use)
AustriaNone supported.AzerbaijanNone supported.BahamasNone supported.BahrainNone supported.BangladeshNone supported.BarbadosNone supported.BelarusNone supported.BelgiumNone supported.BelizeNone supported.BeninNone supported.BermudaNone supported.BhutanNone supported.BoliviaNone supported.Bosnia and HerzegovinaNone supported.BotswanaNone supported.Bouvet IslandNone supported.BrazilNone supported.British Indian Ocean TerritoryNone supported.British Virgin IslandsNone supported.BruneiNone supported.BulgariaNone supported.Burkina FasoNone supported.BurundiNone supported.CambodiaNone supported.CameroonNone supported.Canada
Google Maps (Terms of Use)
Yahoo! Maps (Terms of Use)
MapQuest (Terms of Use)
Cape VerdeNone supported.Cayman IslandsNone supported.Central African RepublicNone supported.ChadNone supported.ChileNone supported.ChinaNone supported.Christmas IslandNone supported.Cocos (Keeling) IslandsNone supported.ColombiaNone supported.ComorosNone supported.Congo (Brazzaville)None supported.Congo (Kinshasa)None supported.Cook IslandsNone supported.Costa RicaNone supported.CroatiaNone supported.CubaNone supported.CyprusNone supported.Czech RepublicNone supported.DenmarkNone supported.DjiboutiNone supported.DominicaNone supported.Dominican RepublicNone supported.East TimorNone supported.EcuadorNone supported.EgyptNone supported.El SalvadorNone supported.Equatorial GuineaNone supported.EritreaNone supported.EstoniaNone supported.EthiopiaNone supported.Falkland IslandsNone supported.Faroe IslandsNone supported.FijiNone supported.FinlandNone supported.FranceNone supported.French GuianaNone supported.French PolynesiaNone supported.French Southern TerritoriesNone supported.GabonNone supported.GambiaNone supported.GeorgiaNone supported.Germany

etc... :(

CommentFileSizeAuthor
#9 location-php53-reroll.patch942 bytesJohn Franklin
#8 location.admin_.options.patch826 bytestangent
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

YesCT’s picture

YesCT CreditAttribution: YesCT commented
Status: Active » Postponed (maintainer needs more info)
Issue tags: +Location theming

Please check to see if this is still a problem in the new release. If it is, reply with a new summary and set the status back to active.

hip’s picture

hip CreditAttribution: hip commented
Status: Postponed (maintainer needs more info) » Active

Still a problem. Same symptoms.

DarkRestraint’s picture

DarkRestraint CreditAttribution: DarkRestraint commented

I am having the same issue. Error code when I go to set up locations.

warning: Parameter 1 to theme_location_map_link_options() expected to be a reference, value given in /home/***/public_html/includes/theme.inc on line 656.

I checked the attributed line in theme.inc and found no reference to that function call. Any and all help would be greatly appreciated.

hutch’s picture

hutch CreditAttribution: hutch commented

I may be wrong but this looks like a bug in

in location.admin.inc
function theme_location_map_link_options(&$form) line 292
and
function theme_location_geocoding_options(&$form) line 313

I cannot see any reason why $form is being passed by reference, I think these should be
function theme_location_map_link_options($form)
and
function theme_location_geocoding_options($form)

Another one to stumble over php 5.3.x stricter checking
You can switch these warnings off in php.ini, they are not errors.

roknrod12’s picture

roknrod12 CreditAttribution: roknrod12 commented

subscribing... same problem here on a local installation.

daniele77’s picture

daniele77 CreditAttribution: daniele77 commented

No problem with PHP versions lower than 5.3.

The problem is caused by the version of PHP 5.3x and later.

Can anyone help me?

hansf’s picture

hansf CreditAttribution: hansf commented

I can verify that removing the & fixed it onthat page.

But on the node view page:
warning: Parameter 2 to location_addanother_form() expected to be a reference, value given in C:\xampp\htdocs\mko\includes\form.inc on line 376. Don' know if this is related.

Php 5.3.1

tangent’s picture

tangent CreditAttribution: tangent commented
Status: Active » Needs review
FileSize
location.admin_.options.patch826 bytes

I also confirm this bug on PHP 5.3. Attached is a patch against HEAD.

John Franklin’s picture

John Franklin CreditAttribution: John Franklin commented
FileSize
location-php53-reroll.patch942 bytes

Prior patch doesn't apply anymore. The parameters were renamed and that broke the patch. Here is a re-rolled version.

rooby’s picture

rooby CreditAttribution: rooby commented
Status: Needs review » Closed (duplicate)

This issue is a duplicate of #733438: Parameter 1 to theme_location_geocoding_options() expected to be a reference, value given (Make compatible with PHP 5.3) and has now been fixed in the current dev version.