Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fatal error with boolean admin UI after using batch option #3936

Closed
Jimmi08 opened this issue Aug 25, 2019 · 3 comments
Closed

fatal error with boolean admin UI after using batch option #3936

Jimmi08 opened this issue Aug 25, 2019 · 3 comments
Assignees
Labels
core: Admin-UI type: bug A problem that should not be happening
Milestone

Comments

@Jimmi08
Copy link
Contributor

Jimmi08 commented Aug 25, 2019

Boolean field defined this way:

			'element_active' => array ( 'title' => LAN_ACTIVE,
			'type' => 'boolean',
			'data' => 'int',
			'width' => 'auto', 
      'batch'=> true,
			'readParms' => array (),
		  'writeParms' => array (),
			'class' => 'left',
			'thclass' => 'left',
			),

Set yes / no on modify Active is working
There 20 records on one page
After using Invert batch option:

Fatal error: Uncaught ArgumentCountError: Too few arguments to function e_admin_ui::handleListBoolreverseBatch(), 2 passed in xxx\e107_handlers\admin_ui.php on line 3444 and exactly 3 expected in xxx\e107_handlers\admin_ui.php:5329 Stack trace: #0 xxx\e107_handlers\admin_ui.php(3444): e_admin_ui->handleListBoolreverseBatch(Array, 'element_active') #1 xxx\e107_handlers\admin_ui.php(4925): e_admin_controller_ui->_handleListBatch('boolreverse__el...') #2 xxx\e107_handlers\admin_ui.php(2094): e_admin_ui->ListBatchTrigger('boolreverse__el...') #3 xxx\e107_handlers\admin_ui.php(1264): e_admin_controller->dispatchObserver() #4 xxx\e107_handlers\admin_ui.php(1074): e_admin_dispatcher->runObservers(true) #5 xxx\e107_plugins\jmelements\admin\admin_config.php(439): e_admin_dispatcher->__construct() #6 {main} thrown in xxx\e107_handlers\admin_ui.php on line 5329

Confirmed with clean generated admin_config by plugin builder (no custom changes, only added batch)

@Moc Moc added core: Admin-UI status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. type: bug A problem that should not be happening and removed type: bug A problem that should not be happening labels Aug 26, 2019
@CaMer0n CaMer0n added this to the e107 2.2.2 milestone Sep 2, 2019
@Moc Moc added type: bug A problem that should not be happening and removed status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. labels Sep 25, 2019
@CaMer0n
Copy link
Member

CaMer0n commented May 14, 2020

@Jimmi08 I now this is a long time ago, but would you remember if the plugin was installed after being generated?

@Jimmi08
Copy link
Contributor Author

Jimmi08 commented May 14, 2020

@Jimmi08 I now this is a long time ago, but would you remember if the plugin was installed after being generated?

It is still there. Elements plugin. Are you able to download it from e107sk.com?

@Jimmi08
Copy link
Contributor Author

Jimmi08 commented May 14, 2020

The same error with news - try to mark more records and use invert on sticky field.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core: Admin-UI type: bug A problem that should not be happening
Projects
None yet
Development

No branches or pull requests

3 participants