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

Glitch in classes/UrlRewrite.php #139

Closed
wants to merge 1 commit into from
Closed

Glitch in classes/UrlRewrite.php #139

wants to merge 1 commit into from

Conversation

Traumflug
Copy link
Contributor

Environment: default installation, then enabled multishop, created
two shops, switched back to monoshop.

Behaviour: some 15 messages similar like this one when accessing
backoffice -> Preferences -> SEO & URLs:

Notice: Undefined offset: 10 in /var/www/html/ThirtyBees/classes/UrlRewrite.php on line 509
Notice: Undefined offset: 10 in /var/www/html/ThirtyBees/classes/UrlRewrite.php on line 509
Notice: Undefined offset: 11 in /var/www/html/ThirtyBees/classes/UrlRewrite.php on line 509
Notice: Undefined offset: 11 in /var/www/html/ThirtyBees/classes/UrlRewrite.php on line 509

Diagnosis: $idCategory is larger than array size of $categories.

Cure: well, the obvious quick fix is to check the array pointer, which is what I did. However, these messages might hint to something actually going wrong, so just removing the message covers the actual problem without solving it.

Environment: default installation, then enabled multishop, created
             two shops, switched back to monoshop.

Behaviour: some 15 messages similar like this one when accessing
           backoffice -> Preferences -> SEO & URLs:

    Notice: Undefined offset: 10 in /var/www/html/ThirtyBees/classes/UrlRewrite.php on line 509
    Notice: Undefined offset: 10 in /var/www/html/ThirtyBees/classes/UrlRewrite.php on line 509
    Notice: Undefined offset: 11 in /var/www/html/ThirtyBees/classes/UrlRewrite.php on line 509
    Notice: Undefined offset: 11 in /var/www/html/ThirtyBees/classes/UrlRewrite.php on line 509

Diagnosis: $idCategory is larger than array size of $categories.
@firstred
Copy link
Contributor

Thanks a lot for your contribution!
We will have a look at it in Monday, when we can take the time to thoroughly check everything.

@firstred firstred added this to the 1.0.1 milestone Mar 10, 2017
@firstred
Copy link
Contributor

Hmmz, haven't been able to reproduce it, yet. That's annoying :/

@Traumflug
Copy link
Contributor Author

Just restored the database I had when I observed this glitch and ... I can't reproduce it either. Too bad.

Looks like I have to wait until it appears again and then try to find the root cause locally. Thanks for listening and sorry for the noise.

@firstred firstred closed this Apr 4, 2017
@Traumflug Traumflug deleted the arrayexhaustion branch April 23, 2017 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants