Hi,
Just installed the dev version, and browsed to Administer › Site configuration › Biblio settings › Fields

Then clicked the secondary tab "Publication Types" (admin/settings/biblio/fields/type) and received this error:

Fatal error: Cannot unset string offsets in /MYSITE/includes/form.inc on line 485
CommentFileSizeAuthor
#1 biblio_473272.patch806 bytesscottrigby

Comments

scottrigby’s picture

Status: Active » Needs review
StatusFileSize
new806 bytes

This seems to do the trick in biblio.module:

   $items['admin/settings/biblio/fields/type'] = array(
     'title'             => 'Publication Types',
-    'page callback'     => 'drupal_get_form',
-    'page arguments'    => array('biblio_admin_types_form'),
+    'page callback'     => 'biblio_admin_types_form',
     'access arguments'  => array('administer biblio'),
     'file'              => 'biblio.admin.inc',
     'type'              => MENU_LOCAL_TASK,

Patch attached.
Any reason why this wouldn't be a good idea btw?

rjerome’s picture

That's perfectly fine. What happened was part of a change in the HEAD branch of the CVS tree inadvertently got copied to the 6--1 branch.

rjerome’s picture

Status: Needs review » Fixed
scottrigby’s picture

cool - thanks :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.