Support has been upgraded!
The Support Forum is closed. Not to worry! Providing the top quality support you expect and we're known for will continue! We're not ending support, just changing where you submit requests. This will provide you with the best experience possible.
Premium Support
Have you purchased an addon for Connections such as one of our premium templates or extensions with a valid license and you need help?
Please open a Support Ticket in your user account.
Free Support
Are you using the free Connections plugin? Don't worry, you are still very important to us! We are still providing you with the same high quality support that we're known for.
Please open a new support topic in the WordPress support forums for Connections.
Tagged: 8.5.22, fatal error. screen options, manage, Toolset Types
- This topic has 5 replies, 2 voices, and was last updated 9 years, 11 months ago by
Jeremy Miller.
-
AuthorPosts
-
08/23/2016 at 7:55 pm #387560
Jeremy Miller
ParticipantHey there!
Love the plugin but I did notice that when attempting to change the number of entries listed in the “Manage” screen, using the “Screen Options” tab (in the admin side) I’m getting:
Warning: Illegal string offset ‘filter’ in /home/snfus/intranet.snf.us/wp-content/plugins/connections/includes/admin/class.functions.php on line 391
Fatal error: Cannot use string offset as an array in /home/snfus/intranet.snf.us/wp-content/plugins/connections/includes/admin/class.functions.php on line 391
So I can’t list more than the default 50 entries per page in the backend. Any help would be appreciated!
08/24/2016 at 12:06 pm #387659Steven Zahm
Keymaster@ Jeremy
Hmmm… looking at the code (which is pretty darned old), it could definitely be a bit more robust in case of errors. I’ve made the necessary changes to hopefully ensure this error is no longer possible. I’m planning on releasing an update on Friday which will include this tweak.
Apologies for the trouble!
ps. Please do let me know if the update does not fix the issue for you.
08/24/2016 at 4:27 pm #387700Jeremy Miller
ParticipantAwesome, thanks for the quick response Steven! I’ll follow up if I run into anything after the patch release.
Thanks!
08/28/2016 at 10:09 am #388048Jeremy Miller
ParticipantHey Steven,
I checked out the latest update and it doesn’t throw a fatal error anymore, so that’s good! The Manage list option still won’t set the entry list to more than 50 on a page though (in the admin section). I did a little troubleshooting and figured out that there’s some sort of conflict with Toolset Types plugin. With that disabled I can change how many entries are listed to whatever I want.
Any thought? Thanks!
08/29/2016 at 11:59 am #388099Steven Zahm
Keymaster@ Jeremy
I pulled down a copy of the plugin from wp.org, did a quick search and found these lines of code:
add_filter('set-screen-option', 'wpcf_table_set_option', 10, 3);
function wpcf_table_set_option($status, $option, $value)
{
return $value;
}Their filter has a bug. It should check for an
$optionname, one of theirs before returning$value. If it is not one of their$optionthey should be returning$status.My guess is that their filter is running after the filter Connections set for its page options, basically corrupting the data. When saving the data, Connections could not properly deal with the “corrupted ” data because it was not getting the data in the expected format. That is what I fixed to make it more resilient.
Now, when the data is read, since it is “corrupt”, Connections already dealt with this by basically resetting the data to the default values if it did not receive the expected format. And, this, is what you are seeing.
There not really much I can do to fix this from my end. The devs for Toolset Types plugin should fix it. The only thing I could try is making my filter higher priority so it runs later (after) the Toolset Types plugin filter.
Hope that explains the issue without making it too technical or complicated sounding!
ps. I decided to increase the priority anyway, so that may help regardless if they fix the bug or not. I’ll include this in the next update.
08/29/2016 at 5:27 pm #388123Jeremy Miller
ParticipantAwesome, thanks Steven! That totally makes sense, I appreciate the help. I’ll go ahead and submit this bug to them. Thanks!
-
AuthorPosts
You cannot reply to this support topic. Please open your own support topic.
