Current CVS: "how to set owner filter default to 'All'

[GoD_AbovE]

Will this work on dp 2.0 and up?

[davidcasey]

I wanted a user default view to remain set to their company, but users in my company (the first company in the database) would default to all.

I am running version 2.1.3.
In /modules/companies/index.php on line 26 I changed to:

PHP Code:

if ($AppUI->user_company == 1 || $owner_filter_id !== 0) {
    $AppUI->setState('owner_filter_id', $owner_filter_id_pre);
} else {
    $owner_filter_id = $AppUI->getState('owner_filter_id', $AppUI->user_id);
} 


If you wanted everyone to default to all, you could change the if conditional to

PHP Code:

if (true || $owner_filter_id !== 0) { 


[darkmatter]

Won't this:

PHP Code:

if (true || $owner_filter_id !== 0) { 


always be true? What's the point of the 'or' part?

[davidcasey]

It does always result true. It doesn't destroy the original code. In case you ever want to do something different, you still have the context to figure out how to achieve other permissions...or to fix it should you ever update the code and overwrite your changes.

If you don't care about that, you could easily remove the entire if statement and replace it with:

PHP Code:

$AppUI->setState('owner_filter_id', $owner_filter_id_pre);