I have a fresh install of drupal 5.12, CCK 5.x-1.10 and Link 5.x-2.5. I created a 'link' cck field for the 'story' content type. I chose 'multiple' when configuring the link field settings, leaving everything else default. If I create a story node, fill in only the 'title', and submit it, two empty records (field_xxx_title and field_xxx_link are empty) are saved to the database table (content_field_xxx) that relates to the link field. It seems that there is no checking being done to ensure that a link's title and url are not empty before saving the data to the database. Now, in my custom template, when I want to only print out urls if a piece of content has them, 2 blank urls are "printed", rather than none.

CommentFileSizeAuthor
#2 link-336426-2.patch549 bytespathfinderelite

Comments

cirotix’s picture

cirotix commented

Hi,

I have the same problem. Have you been able to solved it ?

pathfinderelite’s picture

pathfinderelite commented
StatusFileSize
new link-336426-2.patch549 bytes

I have created a patch which simply does a little extra checking before the URLs are saved. However, I am not sure if this solves all cases of the problem. The issue could probably use some more thorough investigation.

pathfinderelite’s picture

pathfinderelite commented
Status: Active » Needs work
dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented
Status: Needs work » Closed (won't fix)

Issue date expired ...

For any feature request please take a look on the link module versions for Drupal 6 and Drupal 7 with much more features and better server and web standart support since Drupal 5 is not supported no more officially.

The stop for Drupal 5 support: Since we have to catch up to get ready for the final Drupal 7 release and have to deal with many feature requests for the Drupal 6 and Drupal 7 version of the link field module, we sadly have to annouce the stop of Drupal 5 support. The files will stay for any individual use or customization, but any support in the issue queue and in the forums has expired from now on.

Sorry and thanks for understanding.