On a vanilla install, I installed freelinking and freelinking_prepopulate.
On the Freelinking Settings page (/admin/config/content/freelinking), I load the page and save without changing any settings and the following warning is displayed:
An illegal choice has been detected. Please contact the site administrator.
The dblog shows this:
Illegal choice 0 in Advanced Options element.
If I uncheck the "Advanced Options" checkbox, the Freelinking Setting form submits just fine.
I did some variable dumping from the freelinking_prepopulate.module
around lines 110. It looks like the $options
array has one element that is NULL. I suppose that element probably shouldn't be created, or the empty($options)
check needs to change.
... a little GIT blame investigating seems that maybe this bug was in commit #124d6ae.
Comment | File | Size | Author |
---|---|---|---|
#4 | 1689918-list_fields.patch | 4.05 KB | bc |
Comments
Comment #1
xurizaemonEDIT: See below.
You're right, I think that line should be$nodecreate = freelinking_prepopulate_list_fields('nodecreate');
not
$nodecreate[] = freelinking_prepopulate_list_fields('nodecreate');
Comment #2
xurizaemonOK, this was masking some other weirdness ...
freelinking_prepopulate_list_fields('nodecreate')
returns different results on being called a second time. Doing this actually gives labels to the checkboxes (obviously we should fix the underlying issue instead).Comment #3
dooug CreditAttribution: dooug commentedRight. Looks like the debugging/patching for these issues will be happening here: #1665010: Recursion bug in D7 branch when references are circular
Comment #4
bc CreditAttribution: bc commentedthis may or may not be a discrete issue.
in the attached patch i did a few things:
i'll be testing this to see if it has any effect on #1665010: Recursion bug in D7 branch when references are circular
Comment #5
gisleThis whole issue comes from a checkbox to toggle a variable
freelinking_prepopulate_node_advanced
in the freelinking_prepopulate module.Comment #6
gisleLooks like this is a duplicate of #2120353: freelinking prepopulate migration bug from d6 to d7. While this is an older issue the solution in the other thread looks promising, so I am going to base further work on that.