Is there some interest in porting the module to D7 ?

CommentFileSizeAuthor
#11 textformatter-7.x-1.x.patch12.7 KBdamiankloip
#10 textformatter-7.x-1.x.patch12.73 KBdamiankloip

Comments

calte’s picture

calte commented

+1

Scott J’s picture

Scott J commented

I think this would be very useful.

pepemty’s picture

pepemty commented

+1

basicmagic.net’s picture

basicmagic.net commented

subscribe

mlncn’s picture

mlncn commented

This being worked on?

jdufaur’s picture

jdufaur commented

subscribe

damiankloip’s picture

damiankloip commented

I have made a sandbox for the D7 version of this module (http://drupal.org/sandbox/damiankloip/1191818) and made an initial commit. I have been speaking to becw about this and it's up for review. So hopefully there should be a 7.x branch for this project shortly!

skizzo’s picture

skizzo commented

thank you damiankloip! subscribing

jenpasch’s picture

jenpasch commented

thanks, looking forward to having this in the d7 arsenal

damiankloip’s picture

damiankloip commented
Status: Active » Needs review
StatusFileSize
new textformatter-7.x-1.x.patch12.73 KB

Here is a D7 patch (taken from my sandbox ^^).

damiankloip’s picture

damiankloip commented
StatusFileSize
new textformatter-7.x-1.x.patch12.7 KB

Updated patch, an xss fix and a few other things.

becw’s picture

becw commented
Status: Needs review » Fixed

I've reviewed this patch and committed it with a few changes. My most significant edit was to theme_textformatter_comma() -- the 'and' option didn't have enough spaces, and setting the 'html_tag' setting to "none" still yielded a tag. See commit b09c38.

kansaj’s picture

kansaj commented

not quite sure but in the"comma" case the output are not the labels, but the keys. So instead to print "zero" form the pair 0|zero it prints 0, which is not correct.

damiankloip’s picture

damiankloip commented

I don't think this has been tested yet for select lists. Something to look at.

becw’s picture

becw commented

kansaj, damiankloip: I've created a new issue for the bug report in #13 -- #1311466: Printing values rather than labels in comma-separated lists

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.