Closed (fixed)
Project:
Vertical Tabs
Version:
6.x-1.0-rc1
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
7 Nov 2009 at 11:19 UTC
Updated:
22 Jul 2011 at 07:18 UTC
See the attached screenshot. On the content type edit form, I cannot select which fieldsets should be rendered by Vertical tabs. To notice that there is a module that actually add a fieldset in the node edit form.
As side note, I also get the following error message, in the content type edit form:
warning: Parameter 2 to drupal_retrieve_form() expected to be a reference, value given in /Library/WebServer/Documents/dr61/sites/all/modules/vertical_tabs/vertical_tabs.module on line 163.
| Comment | File | Size | Author |
|---|---|---|---|
| Voila_Capture_49.png | 22.96 KB | avpaderno |
Comments
Comment #1
avpadernoI forgot to say I am using the development snapshot from November 7; if the code has been already changed, and I need to wait 1 hour to get the new archive, then I apologize for this report.
Comment #2
avpadernoThe line for which the error is report is contained in the following function:
The line that causes the error I reported contains
$node_form = call_user_func_array('drupal_retrieve_form', $args).As last note, I am having the issue on Mac OS X 10.6, which runs PHP 5.3.0.
Comment #3
dave reidThis is a PHP 5.3 error. I'm going to replace this with a direct call to drupal_retrieve_form($node_form_id, $node_form_state);
Comment #4
dave reidFixed in CVS. Thanks!
Comment #6
fenstratSame php 5.3 error again with rc1
warning: Parameter 2 to drupal_retrieve_form() expected to be a reference, value given in .../vertical_tabs/vertical_tabs.admin.inc on line 45.This happens on admin/content/types/content-type after enabling the option "Expose vertical tabs selection on the edit content type forms."
Comment #7
dave reidMy apologies for the regression. I had used the wrong revision when re-adding the node type form options. I'll get it fixed today.
Comment #8
dave reidFixed back in CVS and will be available in the next release.
http://drupal.org/cvs?commit=326420
Comment #10
bstoppel commentedThis problem still exists with RC1. When will a new RC be available? It's been about a year since the first release candidate.
Comment #11
john franklin commentedThe patch should also remove the $args = array(...) just above the call as it is now dead code. I'd also like to request a new RC (or better yet, full release) be published.
Comment #12
wiifmAlso keen to see a new RC, will save me running more dev code ;)
Comment #13
timb commentedBUMP Dev code fixed the problem for me.
Comment #14
wiciu commentedPHP 5.3.4 (cli) (built: Dec 11 2010 17:46:49)
And still error:
Parameter 2 to drupal_retrieve_form() expected to be a reference, value given w pliku /home/xxx/public_html/modules/vertical_tabs/vertical_tabs.admin.inc, linia 45.
Comment #15
realityloop commentedConfirming dev fixes the issue on PHP 5.3.2
Comment #16
realityloop commentedoops accidentally changed status