Patch attached that simply adds parent::options_form($form, $form_state); to the top of options_form()

This allows views to add all of the other field options into the form. like showing a label, rewriting, wrapping, etc.

Files: 
CommentFileSizeAuthor
#9 privatemsg-views-link-form-1155718-9.patch580 bytesjojonaloha
PASSED: [[SimpleTest]]: [MySQL] 3,157 pass(es).
[ View ]
privatemsg-views-link-form.patch572 bytesJon Pugh
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch privatemsg-views-link-form.patch.
[ View ]

Comments

Status:Needs review» Needs work

The last submitted patch, privatemsg-views-link-form.patch, failed testing.

Berdir’s picture

And do these options actually work? I think I left them out because most made no sense or don't work with the current implementation (which is crap and needs to be fixed, but that's another story)

AntiNSA’s picture

Priority:Normal» Critical

this does not work for me :(....

AntiNSA’s picture

Priority:Critical» Normal

Actually it does work. Thanks

Berdir’s picture

Please provide more information.

What have you tested that works, did you find an issues with some options, ...

Tafa’s picture

works for me

designcontext’s picture

Works for me!
I need to rewrite the field with something.

mjgruta’s picture

This patch is usefull if you want to use "Exclude from display" on your [privatemsg_link] so you can load it on your custom text.

jojonaloha’s picture

Version:7.x-1.1» 7.x-1.x-dev
Issue summary:View changes
Status:Needs work» Needs review
StatusFileSize
new580 bytes
PASSED: [[SimpleTest]]: [MySQL] 3,157 pass(es).
[ View ]

The original patch works for me as well. I was using it to add additional classes in the "Style settings" options.

I've re-rolled the patch for 7.x-1.x-dev, it looks like the original wasn't applying which is why it failed testing.

ptmkenny’s picture