Hi,
i just testing this great module.
But when i click submit on Path page, drupal show this error message:

"Alias Alredy exists" for every textbox present in the page but i haven't alias on this site!

i have attached a screenshot.

Best regards,
Max

CommentFileSizeAuthor
#11 alias_bug_fix.patch8.38 KBbardkerbie
voca-index.jpg113.35 KBcapitano83

Comments

Andrzej7’s picture

I have the same error

xano’s picture

Hey,

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.

xano’s picture

Assigned: Unassigned » xano
Status: Active » Fixed

The 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.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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

Andrzej7’s picture

Status: Closed (fixed) » Active

thanks for an answer, but the bug still exists

regards

Andrzej

xano’s picture

Status: Active » Closed (fixed)

Andrzej7: The bug has been fixed locally and as I have explained in your other issue I will update the download ASAP.

bardkerbie’s picture

Status: Closed (fixed) » Needs work

Still broken. I downloaded the latest commit, installed, and it still complains that "alias already exists".

xano’s picture

Status: Needs work » Closed (fixed)

It 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.

Andrzej7’s picture

one comment :-)

If the bug is still valid, the status of the case should be "active" - not "closed".

xano’s picture

You mean if it's not yet committed?

//Edit: Current release shouldn't contain the bug anymore.

bardkerbie’s picture

Status: Closed (fixed) » Needs review
StatusFileSize
new8.38 KB

Xano, 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.

xano’s picture

Status: Needs review » Fixed

I'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.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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

xano’s picture

Assigned: xano » Unassigned