"Strip HTML tags" option for text handlers [#379382]

Comment	File	Size	Author
#2	views_handler_field_strip_tags.patch	1.42 KB	neochief

	views_handler_field_strip_tags.patch	1.38 KB	neochief

Comment #1

merlinofchaos CreditAttribution: merlinofchaos commented 22 February 2009 at 01:00

I like this quite a bit. In thinking about it, I realized that if this option is set, it forces whatever data there to be an inline element (since it has no HTML tags) -- which means we should force an element_type() override when this is set.

Log in or register to post comments

Comment #2

neochief CreditAttribution: neochief commented 22 February 2009 at 01:28

File	Size
views_handler_field_strip_tags.patch	1.42 KB

Sorry, I not very familiar with views under the hood. Anyway, is this what you talking about?

Log in or register to post comments

Comment #3

seaneffel CreditAttribution: seaneffel commented 6 April 2009 at 22:01

Did this get committed? If not, I've voting to do it. I just tested the top patch and it solved the immediate problem for me.

Log in or register to post comments

Comment #4

merlinofchaos CreditAttribution: merlinofchaos commented 8 April 2009 at 03:51

Status:

Needs review

» Fixed

Committed! Thanks, neochief

Log in or register to post comments

Comment #5

tempd5 CreditAttribution: tempd5 commented 13 April 2009 at 18:16

Apologies for being new to this. Where does this patch go? If I'm reading this correctly (no guarantee there), this will solve http://drupal.org/node/430500, correct?

Log in or register to post comments

Comment #6

seaneffel CreditAttribution: seaneffel commented 13 April 2009 at 19:31

tempd5, yes. I think the problem you are describing was fixed by the patch above. You could do two things:

1. Run this patch against the file in your current Views module folder called "views_handler_field.inc". The first line of the patch tells you the name of the file to patch against.

2. Upgrade to the current dev version of Views? Someone can correct me if I'm wrong, but I think this patch was committed last week even though I don't see it on the release notes of the module.

Also, temp, you had an excellent write up of your issue. I wish everyone could be so articulate on their issues - things would get solved so much faster!

Log in or register to post comments

Comment #7

tempd5 CreditAttribution: tempd5 commented 20 April 2009 at 23:43

Thanks @seaneffel (btw, the patch worked great!)

Now, for what may be the dumbest question ever, how to I upgrade to the latest version of views (or any other module for that matter).

Never really thought about it, but is it okay to just install the new folder over the old one or will that delete/erase the use of any current views I've setup?