Issue with configuration
cameronp - July 11, 2008 - 11:45
| Project: | Media Mover |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Jump to:
Description
Hi There, just got this issue with the configuration, it is applying an array to a #value form , and hence when drupal does its text check on the form contents, it fails because its an array
latest version from cvs, line 2042 in media_mover_api.module.
$form['config'][$verb]= array(
'#type' => 'hidden',
'#value' => array('module' => $configuration->{$verb}->module, 'action' => $configuration->{$verb}->action),
Trying to get my head around how this is used
Thanks,
Cameron
#1
I've just commented out the value, and set it to empty as from what i can tell its not used for anything...
PS have a fair few updates to patch into the latest dev, will put up shortly once ive tested a bit more.
love the module, keep up the good work, just got to get it a bit more stable.
Thanks,
Cameron
#2
Yeah, looking at this, I think that you're right- it's not really needed. I think I added this when i was doing some re factoring, but with the way the prefixing of the form elements works, it's not necessary.
I'm trying really hard to get things totally stabilized for the 1.0 release- it's been somewhat of a toss up between adding functionality and refactoring to clean things up. Any patches you've got, please do post, I'd love to see what things people are running to.
#3
Automatically closed -- issue fixed for two weeks with no activity.