Hi,
Although I'm not a programmer, this sounds as a very nice add-on. I would suggest to do two things:
1. Change the filter name to be: "Node: PHP Snippet" - more intuitive.
2. Add this snippet option on all the other places (Arguments, Sort criteria, etc')

The advantage of this, is that it can be imported/ exported along with the view - so also non programmers, don't need to handle with code programmers give them...

Comments

gnassar’s picture

Status: Closed (fixed) » Closed (won't fix)

Thank you for your comments.

1. The filter naming is <module>:<field>. The field being filtered on in the Node module is the ID field. "PHP snippet" would not be appropriate, I think.

2. This would probably make for a really lousy argument :-) nobody should want to embed PHP in a get field; that's just begging for trouble of the hacker kind. And making it a sort criterion wouldn't work either; Node: ID is already a sort criterion. The only way I can think of to do that is to create a temp table for display purposes with an additional PHP-generated field to sort by, and sorting on that secondary field -- read: nightmare.

amitaibu’s picture

Status: Active » Closed (fixed)

Thanks man for the reply. Cheers.

Status: Closed (won't fix) » Closed (fixed)