Closed (fixed)
Project:
Views (for Drupal 7)
Version:
7.x-3.x-dev
Component:
node data
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
19 Jun 2009 at 22:45 UTC
Updated:
13 Jul 2010 at 08:41 UTC
Jump to comment: Most recent file
Comments
Comment #1
dawehnerneeds review :)
Perhaps there should be a easier mechanism to extends fields for things like this.
Comment #2
michelleHey, thanks. Earl said to just put it in the style so that's what I did but I will give this a shot and see if it works better. That will make it more flexible for folks as well to decice if they want the sticky label there or not.
Michelle
Comment #3
merlinofchaos commentedI would rather modify the boolean handler so that fields can add custom variations without writing new handlers.
Comment #4
merlinofchaos commentedComment #5
dawehnerI'm wondering myself how this can be achived
I guess it could be done in the hook_views_data definition
Perhaps with a additional parameter here, i see i have to read more in views.
Comment #6
merlinofchaos commentedYeah. It'd be something like 'output formats' => array('sticky' => array('', t('Sticky'))
Or something.
Comment #7
dawehneryeah, thats much more flexible
Comment #8
merlinofchaos commentedThis patch looks good. 'output formats' needs to be documented in the header of the class, since that's the only place to find documentation for what's available in the definitions.
Comment #9
dawehnerUpdate
Comment #10
merlinofchaos commentedFor documenting definition keys, I usually use doxygen's list formatting capabilities, so each item should have a - in front of it. Since you've got an array embedded, it probably also needs @code and @endcode tags.
Comment #11
dawehnerHere is a patch which uses the codestyle of definitions. I'm not sure about the array
Comment #12
merlinofchaos commentedDoesn't seem to apply to 7.x, but applied to both 6.x branches.
Actual patch applied attached.
Comment #13
dawehnerThanks for making the patch ready... now i will be able to use this syntax somewhere else .
I commited the patch to d7.
Comment #15
dawehnerThis is now really commited.
Comment #16
mastoll commentedI like this option for Node:sticky. How would one apply it to Node:published as well?