Hi,

Just upgraded to location 5.x-3.0

No errors or warnings on update.php after install
activated user location module.
all appropriate permissions are still active from previous version

Tried to add new users
Tried to edit the location information in my user account
No luck. In both cases get

Fatal error: Cannot use string offset as an array in /home/groupsal/public_html/sites/all/modules/location/location.module on line 527

also, whenever I visit an administrative page (ex modules, status, etc) I get a pink warning box that says...

warning: array_merge() [function.array-merge]: Argument #1 is not an array in /home/groupsal/public_html/sites/all/modules/location/location.module on line 1385.

I am using these additional contrib modules.
usernode
nodeprofile (and related required supporting modules)
advanced profile

I have location collection active on two content types
user profile.
usernode.

I can successfully edit the location fields associated with the user profile, but not those associated with the user account (and the usernode that mirrors it)

any ideas, patches, workarounds? It's a show stopper, but I haven't seen anyone else talking about this one. So I'm guessing it must be operator error... Anyone else experience anything like this?

Thanks

FWIW,

Drupal 5.12
Configuration file Protected
Cron maintenance tasks Last run 5 weeks 1 day ago
You can run cron manually.
Database schema Up to date
Drupal core update status No update data available
No information is available about potential new releases for currently installed modules. To check for updates, you may need to run cron or you can check manually. Please note that checking for available updates can take a long time, so please be patient.
File system Writable (public download method)
JQuery Update Installed correctly
The current installed version of JQuery is 1.1.2
MySQL database 5.0.67
Organic groups modules Organic groups works best when job_queue.module.module is enabled. See the Integration section of the README file.
PHP 5.2.8
PHP register globals Disabled
Unicode library PHP Mbstring Extension
Web server Apache/2.2.11 (Unix) mod_ssl/2.2.11 OpenSSL/0.9.8b mod_bwlimited/1.4

Comments

jackspiv’s picture

Priority: Normal » Critical

just upgrade to critical. users can't update their profiles. having to remove registration links for new users.
seemed to work in sandbox, so its now on production site. not anxious to rollback.

Ideas? Anyone?

bdragon’s picture

Assigned: Unassigned » bdragon

Looking into it.

bdragon’s picture

Status: Active » Postponed (maintainer needs more info)

Ahh yes.
Looks like locpick isn't necessarily sane if users aren't allowed to set the user location.
We shouldn't even be attempting to validate in that case.

http://drupal.org/cvs?commit=164289

(forward ports)
http://drupal.org/cvs?commit=164291
http://drupal.org/cvs?commit=164292

Please check if this fixes your crash.

Additionally, the warning you mentioned was fixed already in CVS, and 3.1 will have the fix, which should hopefully be out later today.

jackspiv’s picture

Awesome dude! Thanks.
It works. And saves me a serious headache.

More specifically,
it works...

1) on a 5.x-3.0 location file already patched per http://drupal.org/node/346988 (don't know if you assumed that or not for your 3.1 release)

2) the patch fails for some reason on the change to the comment line at the head of the file. I just edited that by hand.

bdragon’s picture

Status: Postponed (maintainer needs more info) » Closed (duplicate)