Drop Task: Add a link component to flexifilter module

Patch fails agains 6.x-1.1-rc1:

$ patch -p0 < flexifilter_link_01.patch
patching file flexifilter.components.inc
Hunk #2 FAILED at 666.
Hunk #3 FAILED at 828.
2 out of 3 hunks FAILED -- saving rejects to file flexifilter.components.inc.rej

patching file flexifilter.flexifilters.inc

Applicable to CVS HEAD, please.

Minor bug:
The delimiter doesn't appear by default.

bigger bug:

original content
--------------
[[hello|hello]]

[hello|hello]

filtered content (appears as link)
--------------

hello]]</p> <p>[hello|hello]</p>

export
-------
a:8:{s:5:"label";s:4:"Test";s:11:"description";s:11:"just a test";s:2:"id";s:1:"2";s:7:"enabled";b:1;s:8:"advanced";b:0;s:5:"delta";s:1:"1";s:10:"components";a:3:{i:0;a:3:{s:5:"class";s:26:"flexifilter_link_component";s:8:"settings";a:6:{s:8:"ordering";s:1:"0";s:7:"divider";s:1:"|";s:8:"no_links";i:0;s:10:"check_path";i:0;s:6:"prefix";s:0:"";s:4:"step";s:7:"process";}s:2:"id";s:2:"94";}s:7:"id_next";i:95;s:9:"id_prefix";s:22:"flexifilter_component_";}s:3:"fid";s:3:"new";}

You need to wrap it within a chunk grabber for anything useful to happen. What's happening here looks like a strange combination of a naive use of flexifilter with the line break filter.

The minor bug is still there - "The delimiter doesn't appear by default" (i.e. I get a drupal error message "Divider field is required.")

Another strange issue I get, although I must admit I'm still somewhere on the learning curve of this module :)

original preview:
this is a '''test'''

Component: Linkifier:
<a href="/D6/?q=this%20is%20a%20%3Cstrong%3Etest%3C/strong%3E">this is a &lt;strong&gt;test&lt;/strong&gt;</a>

attached flexifilter export.

Amitaibu, for #7: The double quotes around 'test' are apparently being used to give the text a bold layout using the HTML 'strong' tag. So another filter (flexifilter component?) is interfering here.

This is the explanation:

In URL encoding (used in the href), the html brackets < and > are encoded as %3C and %3E. In HTML encoding (used between the 'a' tags), they are encoded as < and >. Similarly, spaces are URL encoded as %20.

Here is a modified version of the patch in #1 above.

Changes:

• Applicable to 6.x-1.1-rc2 (latest released version)
• Bugfix: form default values did not appear (the minor bug in #4 above)
• Bugfix: characters in URL were not urlencode()d, giving html syntax errors for links containing double quotes.
• Feature: integration with pathauto module trough extra option 'pathauto cleanup' (only available when pathauto module is active). This will transform the link to conform to pathauto's rules.
• Code style: replaced nested if block with one if ( x && y) block.

Still to investigate:

• Cannot set default value TRUE for a checkbox, since isset() always returns false when the user de-selects it, and then the default value is applied once again. Result: default value TRUE makes it impossible to set it to FALSE.
• Chunk grabber does not do a good job at parsing chunks that contain chunk delimiters in their content. Case in point: "[link [1] before [otherlink]".
• Cross-site scripting: investigate if there is no vulnerability

This component is now in use on an experimental Drupal install of mine, using the following filter: