"A link with that URL already exists."

Need to check this. I put code in that is the node is the same to skip. I guess I need to a little more checking.

Thanks
Robert

I am looking in that and should have a fix but the end of today. Validation was just added so it is possible that you have two copies of the same link in you DB already. For a quick fix I am making the Validation optional.

Thanks
Robert

Robert, there is a typo in that code: A missing "s" in the variable name on the query.

  $url_exits_nid = db_result(db_query("SELECT nid FROM {weblinks} l WHERE l.url='%s'", $node->url));
  if ($url_exists) {
    if ($node->nid != $url_exists_nid) {

Okay, I just did some debugging and here's what I noticed: A new node (which should have dupe checking) has no "nid" at validation time. An updated node will already have a "nid".

Not that simple I wonder how this worked when I tried it and it worked?

jupiterchild,

You are using drupal 5 correct. Is so replace weblink.module with the one attached here and see if the issue is fixed?

NancyDru,

I fixed the typo in the 6 code. and commited to cvs for 6. 5 code didn't have a typo but also had $node in it not the form_state like it should have in 5.x land.

Thanks
Robert

PS. I still intend to make the validation code optional so people who want dupes or have a lot of dupes already in there DB can turn it off.

This version was about making the dupe check work right. I will add a interface and add the ability to make it optional before the next release.

Thanks
Robert

I think this is two important to wait for me to add the option boxes going to release before the options are done. Marking the issue as fixes some the dupe code is working.