Support

Search results for ""

Sorry, no results found. Perhaps you would like to search the documentation?
All Topics
Tim

ACP 7.0.4 – 7.0.8 cause fatal error when view GravityForm entires

When we run versions 7.0.4 – 7.0.8 of Admin Columns Pro, we get a fatal error when trying to view any of the “Entires” screens for Gravity Forms. For example, this URL causes the error:

https://mydomain.com/wp-admin/admin.php?page=gf_entries

Error Details
=============


An error of type E_ERROR was caused in line 44 of the file
/nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/addons/gravityforms/classes/Field/Field.php.
Error message: Uncaught TypeError:
ACA\GravityForms\Field\Field::is_required(): Return value must be of
type bool, string returned in
/nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/addons/gravityforms/classes/Field/Field.php:44
Stack trace:
#0 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/addons/gravityforms/classes/Editing/EntryServiceFactory.php(117):
ACA\GravityForms\Field\Field->is_required()
#1 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/addons/gravityforms/classes/Editing/EntryServiceFactory.php(58):
ACA\GravityForms\Editing\EntryServiceFactory->set_default_view_properties(Object(ACA\GravityForms\Field\Type\Input),
Object(ACP\Editing\View\Text))
#2 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/addons/gravityforms/classes/Editing/EntryServiceFactory.php(47):
ACA\GravityForms\Editing\EntryServiceFactory->create_view(Object(ACA\GravityForms\Field\Type\Input))
#3 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/addons/gravityforms/classes/ColumnFactory/EntryFactory.php(73):
ACA\GravityForms\Editing\EntryServiceFactory->create(Object(ACA\GravityForms\Field\Type\Input))
#4 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/classes/Column/ColumnFactory.php(29):
ACA\GravityForms\ColumnFactory\EntryFactory->get_editing(Object(AC\Setting\Config))
#5 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/classes/Column/AdvancedColumnFactory.php(35):
ACP\Column\ColumnFactory->get_feature_settings_builder(Object(AC\Setting\Config))
#6 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/admin-columns/classes/ColumnRepository/EncodedData.php(73):
ACP\Column\AdvancedColumnFactory->create(Object(AC\Setting\Config))
#7 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/admin-columns/classes/ColumnRepository/EncodedData.php(46):
AC\ColumnRepository\EncodedData->find('field_id-1.3',
Object(AC\Setting\Config))
#8 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/admin-columns/classes/ColumnIterator/ProxyColumnIterator.php(27):
AC\ColumnRepository\EncodedData->find_all()
#9 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/admin-columns/classes/ColumnIterator/ProxyColumnIterator.php(58):
AC\ColumnIterator\ProxyColumnIterator->forward()
#10 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/admin-columns/classes/ColumnRepository/Sort/ColumnNames.php(25):
AC\ColumnIterator\ProxyColumnIterator->rewind()
#11 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/admin-columns/classes/Service/ManageHeadings.php(45):
AC\ColumnRepository\Sort\ColumnNames->sort(Object(AC\ColumnIterator\ProxyColumnIterator))
#12 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/admin-columns/classes/Service/ManageHeadings.php(71):
AC\Service\ManageHeadings->get_column_headings(Object(AC\ListScreen))
#13 /nas/content/live/thehiredguns/wp-includes/class-wp-hook.php(324):
AC\Service\ManageHeadings->handle(Object(AC\ListScreen),
Object(ACA\GravityForms\TableScreen\Entry))
#14 /nas/content/live/thehiredguns/wp-includes/class-wp-hook.php(348):
WP_Hook->apply_filters(NULL, Array)
#15 /nas/content/live/thehiredguns/wp-includes/plugin.php(517):
WP_Hook->do_action(Array)
#16 /nas/content/live/thehiredguns/wp-content/plugins/admin-columns-pro/admin-columns/classes/Service/CurrentTable.php(99):
do_action('ac/table/list_s...', Object(AC\ListScreen),
Object(ACA\GravityForms\TableScreen\Entry))
#17 /nas/content/live/thehiredguns/wp-includes/class-wp-hook.php(324):
AC\Service\CurrentTable->handle(Object(WP_Screen))
#18 /nas/content/live/thehiredguns/wp-includes/class-wp-hook.php(348):
WP_Hook->apply_filters(NULL, Array)
#19 /nas/content/live/thehiredguns/wp-includes/plugin.php(517):
WP_Hook->do_action(Array)
#20 /nas/content/live/thehiredguns/wp-admin/includes/class-wp-screen.php(424):
do_action('current_screen', Object(WP_Screen))
#21 /nas/content/live/thehiredguns/wp-admin/includes/screen.php(243):
WP_Screen->set_current_screen()
#22 /nas/content/live/thehiredguns/wp-admin/admin.php(213): set_current_screen()
#23 {main}
  thrown

If we got back to version 7.0.3, the issue goes away.

1 day, 17 hours ago
Tobias Schutter
Developer

Thanks for reporting this issue. The GF_Field->isRequired() method in Gravity Forms is expected to return a boolean, but it appears this is not the case on your installation. To address this, we will explicitly cast the value to a boolean in our code. This fix will be included in the next maintenance release V7.0.9.

1 day, 17 hours ago
Tim

Thank you, Tobias.

1 day, 14 hours ago

You must be logged in to reply to this topic.