#3641 - Smart topic notification settings

By Adam Edington
Added 2nd Jul 2018, 4:04 PM
0 views

Identifier	#3641
Issue type	Trivial issue (does not break functionality)
Title	Smart topic notification settings
Status	Completed
Handling member	Chris Graham
Addon	core_fields
Description	Editing the admin profile and putting a check in the Smart topic notifications box appears to do nothing. Profile defaults to No.
Steps to reproduce
Funded?	No

The system will post a comment when this issue is modified (e.g., status changes). To be notified of this, click "Enable comment notifications".

Rating

Unrated

Comments

By Guest posted 3rd Jul 2018, 12:29 PM

I think you probably have added a lot of CPFs and the PHP max_input_vars setting is being exceeded. I have made some tweaks to take the edge of it in
https://github.com/ocproducts/composr/commit/103895580d8da1fd929a6bf653a63c3f165b58c3

Try raising your max_input_vars setting via the .htaccess file (our default raises it to 2000).

By Guest posted 4th Jul 2018, 2:10 AM

I added 3 CPF's tied to a Group, but I will check on that setting to see what Bitnami Wampstack have set it to.

By Guest posted 5th Jul 2018, 1:41 PM

php_value max_input_vars 2000
php_value suhosin.get.max_vars 2000
php_value suhosin.post.max_vars 2000
php_value suhosin.request.max_vars 2000

added to .htaccess file and still no change to that setting when checked and saved, this is without the fixes being applied.

By Guest posted 8th Jul 2018, 12:27 PM

Can you catch me in IRC? This one is stumping me.

By Guest, By Guest, posted 16th Jul 2018, 10:23 PM

Investigation shows that the cns_get_all_custom_fields_match/cns_get_all_custom_fields_match_member functions need to take/default-to null for $special_start, and then accept both positive and negative checks when set to a binary value.

For this change to be made all calls to the functions will need to be assessed.

#3641 - Smart topic notification settings

Rating

Comments

Statistics