Closed (fixed)
Project:
Vocabulary Index
Version:
5.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Oct 2007 at 10:21 UTC
Updated:
23 Jul 2013 at 23:55 UTC
Jump to comment: Most recent file
Comments
Comment #1
Andrzej7 commentedI have the same error
Comment #2
xanoHey,
Are you sure any random input is causing this bug? I'm going to check this out ASAP. I guess there's a bug in the query or so.
Apologies for responding so late. Thought issues of own modules would show up in the tracker or so. Apparently they don't.
Comment #3
xanoThe problem has been fixed. I will upload a new version of this module sometime this week. As I was working on a new version with some improved and extra functionality I decided to ship this fix with it.
The reason for the problem was that I had used drupal_lookup_path with the wrong arguments which resulted in an error no matter what value you entered in te form.
Comment #4
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #5
Andrzej7 commentedthanks for an answer, but the bug still exists
regards
Andrzej
Comment #6
xanoAndrzej7: The bug has been fixed locally and as I have explained in your other issue I will update the download ASAP.
Comment #7
bardkerbie commentedStill broken. I downloaded the latest commit, installed, and it still complains that "alias already exists".
Comment #8
xanoIt is still broken, yes. I'm still figuring our how to work with CVS.
And please don't mess with the issue status if you don't know what you're doing.
Comment #9
Andrzej7 commentedone comment :-)
If the bug is still valid, the status of the case should be "active" - not "closed".
Comment #10
xanoYou mean if it's not yet committed?
//Edit: Current release shouldn't contain the bug anymore.
Comment #11
bardkerbie commentedXano, I don't mean to harsh your mellow, but DRUPAL-5 branch of CVS repo still contains the bug on line 58. I am referring to CVS rev 1.1.2.3.2.1. You're not pushing changes to the DRUPAL-5 branch properly. I don't hold it against you; CVS is not what I'd call user-friendly. I'm reopening the issue to reflect the fact that the 5.x-1.x-dev branch is still broken. I'm also submitting a patch in the hopes that CVS actually takes the changes this time.
Everyone else: If you're feeling courageous, you can fix the bug yourself by changing the FALSE on line 58 to TRUE. You can also apply the attached patch to vocabindex.module.
Comment #12
xanoI'm terribly sorry, this was my fault. That line contained two errors. The first one was that I hadn't used drupal_lookup_path() in the right way and the second was that the result was compared to FALSE rather than TRUE. The first bug was fixed a while ago, but the second wasn't. I fixed it a few minutes ago and committed the changes to CVS. A downloadable package should appear within a few hours.
Thanks for the input people and again my apologies for this rather dumb mistake.
Comment #13
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #14
xano