Really doesn't need any changes besides the .info file...

CommentFileSizeAuthor
#5 textile.info.txt169 bytesdrewish
#4 textile_0.info169 bytesdrewish
textile.info147 bytesdrewish

Comments

gtcaz’s picture

gtcaz commented

I was able to install the module and post some test content to verify basic functionality of this patch.

chrisada’s picture

chrisada commented
Status: Needs review » Reviewed & tested by the community

Module does work with 5.0 RC1 after the info file is added.

colm_iqc’s picture

colm_iqc commented

Any timeframe on getting this released into an actual patch?

drewish’s picture

drewish commented
StatusFileSize
new textile_0.info169 bytes

maybe you'll be able to download this...

drewish’s picture

drewish commented
StatusFileSize
new textile.info.txt169 bytes

okay, i guess not

appel’s picture

appel commented

What's the current status?

appel’s picture

appel commented

Latest HEAD with the above .info patch seems to work in my 5.1 installation (upgrade from 4.7.5). Found one peculiar issue though; if your node body starts with a list and the first list-item on that list contains a link, then it will not be textiled.

This works:

lorem ipsum

* "lorem ipsum":/node/15
* "lorem ipsum":/node/15

This also works:

* lorem ipsum
* "lorem ipsum":/node/15

This won't work:

* "lorem ipsum":/node/15
* "lorem ipsum":/node/15
joshk’s picture

joshk commented

Appel: is that something that worked in 4.7.5, but not in 5.1? If not, it might be a bug in the textile lib itself.

appel’s picture

appel commented

I just checked it, it works in Drupal 4.7.5 with Textile 4.7.x-1.x-dev, so it appears to be a bug with the module. Should we move this into a separate issue?

drewish’s picture

drewish commented

appel, yeah this just adds an info file. there's probably something else that's to blame.

appel’s picture

appel commented

Okidoki, posted a new issue here.

jhriggs’s picture

jhriggs commented
Assigned: Unassigned » jhriggs
Status: Reviewed & tested by the community » Fixed

I just committed version 5.x-1.0 to CVS.

I have not seen the link-as-first-list-item issue. When I created a page with a list as the first thing in the body and the first list item was a link, it worked fine. Please check to see if other filter modules may be conflicting.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)