? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
2 Nov 2017

Pull Request for Issue #18468 .

Summary of Changes

fix Warning: str_repeat(): Second argument has to be greater than or equal to 0

Testing Instructions

see #18468

avatar alikon alikon - open - 2 Nov 2017
avatar alikon alikon - change - 2 Nov 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Nov 2017
Category Administration com_menus
avatar brianteeman brianteeman - test_item - 2 Nov 2017 - Tested successfully
avatar brianteeman
brianteeman - comment - 2 Nov 2017

I have tested this item successfully on f31bb1b


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18471.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 2 Nov 2017

didn't get Failure, where to "enable full php error reporting"?

System information

3.8.2-dev
Multilanguage Site
macOS Sierra, 10.12.6
Firefox 56 (64-bit)

MAMP 4.1.1

  • PHP 7.0.15
  • MySQLi 5.6.35
avatar brianteeman
brianteeman - comment - 2 Nov 2017

Set error reporting to development in global config (it should be at that whenever you are testing)
Then make sure the filters are set exactyl as in the image

avatar csthomas
csthomas - comment - 2 Nov 2017

I understand that this PR hides warning, but only one menu item should has level=0, its Root.

I can not reproduce this warning on fresh joomla.

If we hide that error by this PR then we can not get any information that our/your table #__menu is corrupted. probably this is the wrong approach. I have changed my mind.

avatar infograf768
infograf768 - comment - 2 Nov 2017

I confirm that I do not get this error on a clean staging.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 2 Nov 2017

@brianteeman Error reporting was set on "Maximum", select "Development" didn't get Failure too.

System information

3.8.2-dev
Multilanguage Site
macOS Sierra, 10.12.6
Firefox 56 (64-bit)

MAMP 4.1.1

  • PHP 7.0.15
  • MySQLi 5.6.35
avatar csthomas
csthomas - comment - 2 Nov 2017

@brianteeman You should click the rebuild button, which will repair #__menu table.
After that you should not see more warning.

avatar brianteeman
brianteeman - comment - 2 Nov 2017

nope it is still there

avatar csthomas
csthomas - comment - 2 Nov 2017

I suspect you have incorrect column parent_id too.

You can try to find the smallest set of rows (by filters) that generates warning and then open and save each one. This should fix parent_id.

Other way, could be danger, you can go to phpmyadmin and look for rows with parent_id = 0, (except 'Root') and change it to 1

avatar brianteeman
brianteeman - comment - 2 Nov 2017

screenshotr10-57-19

avatar csthomas
csthomas - comment - 2 Nov 2017

Above rows with id 497 and 498 have wrong parent_id. I'm curious when/why they are corrupted.

avatar brianteeman
brianteeman - comment - 2 Nov 2017

opensing and resaving fixed the parent_id - rebuild did not

avatar csthomas
csthomas - comment - 2 Nov 2017

Can you precise Joomla version or ~date when above rows was created, with parent_id = 0.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 12 Nov 2017

reminder for @brianteeman


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18471.

avatar csthomas csthomas - test_item - 6 Jun 2018 - Tested successfully
avatar csthomas
csthomas - comment - 6 Jun 2018

I have tested this item successfully on f31bb1b

Code review


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18471.

avatar franz-wohlkoenig franz-wohlkoenig - change - 6 Jun 2018
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 6 Jun 2018

Ready to Commit after two successful tests.

avatar mbabker mbabker - change - 12 Jun 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-06-12 22:58:58
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 12 Jun 2018
avatar mbabker mbabker - merge - 12 Jun 2018

Add a Comment

Login with GitHub to post a comment