Closed (duplicate)
Project:
Faceted Search
Version:
6.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 Aug 2010 at 10:33 UTC
Updated:
9 Jul 2013 at 18:12 UTC
Using Drupal 6.17 and Faceted Search 6.x-1.0-beta2
While creating a new Environment, only the name, title and description gets saved to database. If the other required fields are deliberately left blank then error message is shown but if they are populated, their values are not saved.
After some debugging, I noticed that $env->settings has only three keys - title, ignore_status and types after form is submitted. And there's a code in module that checks for existence of a key before adding value to it
foreach ($form_state['values'] as $key => $value) {
if (isset($env->settings[$key])) {
$env->settings[$key] = $value;
}
}
so other keys are simply skipped. There's a comment above this code that suggests the code should work since every setting will have a default value. But this doesn't really seem to be the case.
Comments
Comment #1
kenorb commentedIn my case Environment is empty after Save.
Reported here: #884558: Environment is blank on environment list
Comment #2
steveparks commentedJust to let you know that this issue appears to have been fixed in 6.x-1.x-dev since about the 11th July 2010.
So - beta2 is broken, use 1.x-dev if you want to have a working version of this module
Comment #3
brazorf commentedSubscribe
It's just me or this bug makes the module unusable ?
Comment #4
anavrin commentedI have exactly the same problem ... Environment name all is empty after I save it. Is there any workaround - using dev version is a good idea? On the other hand there is so many installations using beta2 version that it's impossible that this bug occurs in all installations.
I have php 5.3.3-7- on Debian squeeze1, with mysql 5.1.49-3. Maybe all the working installations use php in version 5.2 and older?
Comment #5
rafaelcaceres commentedWorks for me: http://drupal.org/node/913558
Comment #6
ashedryden commentedMoving fron beta-2 to 1.x-dev fixed this issue for me.
Comment #7
david lesieur commentedMarking as a duplicate of #913558: Error with PHP 5.3: Parameter 1 to comment_nodeapi() expected to be a reference.