Closed (won't fix)
Project:
Link
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Mar 2009 at 14:24 UTC
Updated:
20 Dec 2019 at 11:06 UTC
Jump to comment: Most recent
In link_field()'s 'presave', _link_process() assumes the 'attributes' has been unserialized on 'load'. However, this is now always true.
Attached a patch which adds a check.
| Comment | File | Size | Author |
|---|---|---|---|
| link-dont_assume_unserialized.patch | 706 bytes | AmrMostafa |
Comments
Comment #1
ellen.davis commentedI have found the same issue, but in a different part of the code. The _link_sanitize function also assume attributes has been unserialized. This is true when editing, but not when viewing. The same code from this patch needs to be included before the Add attributes section in _link_sanitize.
Testing done using drupal 6.10 and link-DRUPAL-6--1 from cvs.
Comment #2
digi24 commentedI can confirm the need for the patch in the first posting, I do not know about #1, but it does not seem to do any harm for me.
Comment #3
damienmckennaThe Drupal 6 version of this module is no longer supported.