Closed (fixed)
Project:
Node Type Filter
Version:
6.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Anonymous (not verified)
Created:
15 Mar 2008 at 14:22 UTC
Updated:
18 Jan 2019 at 10:35 UTC
Hi,
if I enable Node_Type_Filter, and then try alter content in content admin section (/admin/content/node), like publish/unpublish, delete, etc., I got "An illegal choice has been detected. Please contact the site administrator." error. If Node_Type_Filter is disable, these actions can be made without any error.
Comments
Comment #1
nancydruI will look into this, but the module shouldn't even be at play there.
Comment #2
mooffie commented(I'm ariving here by a post on the forum.)
I'm not using this module, but I had a quick look and I see that it uses $_REQUEST['type']. I believe this should be changed to $_GET['type'] or else --if the form happens to have a 'type' field already, which might very well be the case for the 'admin/content/node' form-- weird things could happen.
Comment #3
nancydruThank, Mooffie.
Comment #4
nancydruI was able to reproduce the problem, and Mooffie's suggestion worked. I would never have found that, so Mooffie saved me again.
Strange that this did not happen in 5.x.
Comment #5
john.karahalis commentedGood call, mooffie. Thanks alot.
Comment #6
john.karahalis commentedI just installed the 6.x-dev branch (which changes $_REQUEST to $_GET), but the bug remains. It's doesn't seem to have anything to do with the form_build_id value as I originally thought, and it seems that it doesn't have anything to do with $_REQUEST either.
I've tried dis/re-enabling the module but even after that the error message returns.
Comment #7
nancydruHmm, I was able to reproduce the problem and it went away after the fix. Did you look at the code to make sure you got the change?
Comment #8
john.karahalis commentedYes the code did change, and $_GET is now being used but the bug remains. For some reason, I am able to delete content but nothing else.
Comment #9
nancydruHmm. I'll do some more digging. I was able to reproduce the problem and was able to do everything after fixing it.
Comment #10
john.karahalis commentedHi. Just letting everyone (especially nancyw) know that the new version -does- now work for me. I don't know why it continued to act buggy for a while after being updated. Maybe it had something to do with caches or some other obscure Drupal tomfoolery 8)
Anyway, it's working now, and it does seem to be as a result of the $_REQUEST fix. Good work everyone and thanks alot.
Comment #11
nancydruGreat, thanks for the update.
Comment #12
john.karahalis commentedupdate:
Now it's not working again. Very strange. I have tried uninstalling/reinstalling the module and still have the problem.
Comment #13
nancydruComment #14
john.karahalis commentedI saw that you just changed the status to fixed. It turns out I am still having the problem after all. I don't know why the bug subsided briefly, but it's back.
Comment #15
nancydruI cahnged it to fixed when you said it was working. When you then said it was not, I changed it to PCNW.
Comment #16
nancydruI just did this with multiple nodes, using "publish," "unpublish,", "promote," and "delete." They all worked fine.
Are you using an access control module or something else that also uses hook_db_rewrite_sql?
Comment #17
john.karahalis commentedI previously had node_type_filter installed in my .../sites/all/modules/node_type_filter directory. Despite updating the module in that directory, the bug remained.
I just realized that .../sites/all/taxonomy_browser contains its own copy of /node_type_filter, presumably the version that still had the bug.
Hopefully if I update the .../taxonomy_browser/node_type_filter directory with the new version of the module, the bug will be fixed completely. Right now I am having hosting issues and so cannot test this, but I will report back shortly.
Comment #18
nancydruDid you put that extra copy there, or did it somehow get distributed that way?
Comment #19
john.karahalis commentedI'm fairly certain I put that copy there accidentally.
I completely removed Taxonomy Browser and Node Type Filter (both copies), and then reinstalled each. Still receiving the error.
I wouldn't be too concerned though. It seems that the fix is working for everyone else. More than likely, I am either doing something wrong or broke Drupal in some way (maybe with another module or theme). I'm more than willing to help you in whatever way possible, but like I said it seems that this bug is now unique to me.
Comment #20
nancydruWell, I have no idea how many "everybody else" is - as far as I know you and I may be the only ones using it. So that makes you important.
So let's start with the $Id line in the module. And verify that this change got made to your current copy, please.
My code shows v 1.2.2.1.2.2.2.2 2008/03/21.
Comment #21
john.karahalis commentedSuccess!!!
Apparently I have been using v 1.2.2.1.2.2.2.1 2008/01/25 19:32:14 during my testing today. This is the version which is available for download on the module's page, http://drupal.org/project/node_type_filter.
After manually changing $_REQUEST to $_GET, everything now works perfectly!
It seems that I was encountering two problems here, $_REQUEST and the multiple copies of the module. A couple of weeks ago I fixed $_REQUEST, but that edit was apparently reversed when I reinstalled the module (unknowingly to 1.2.2.1.2.2.2.1). It wasn't until just now that BOTH problems were corrected. As such, it wasn't until just now that the problem was completely resolved.
Thanks for keeping in contact with me and working with me to get this issue resolved. Keep up the great work...and fix that link!
"given enough eyeballs, all bugs are shallow"
-Eric S. Raymond, Open Source Software advocate
Comment #22
nancydruOh, I guess I wasn't clear. When I make changes, they are always to the -dev version. After I've made a few changes and people have tested them, then I roll up a new official version. Please try the -dev version.
Comment #23
john.karahalis commentedYes, the -dev version seems to work great. Sorry about the confusion.
Comment #24
nancydruGreat.
Comment #25
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.