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

New banner image sort order not displaying. #3510

Closed
bhavin30 opened this issue Oct 16, 2015 · 3 comments
Closed

New banner image sort order not displaying. #3510

bhavin30 opened this issue Oct 16, 2015 · 3 comments

Comments

@bhavin30
Copy link

Notice: Undefined index: sort_order in \opencart-2.1.0.1\upload\admin\controller\design\banner.php on line 386
Save with out adding banner title and get notice.
fresh_opencart_issue

@jremes-foss
Copy link
Contributor

This is yet to be reproduced, but my main suspicion is that the code causing this to happen is this piece in admin/controller/design/banner.php:

            foreach ($banner_image_description_query->rows as $banner_image_description) {
                $banner_image_description_data[$banner_image_description['language_id']] = array('title' => $banner_image_description['title']);
            }

Basically, the foreach loop in question saves the title to array by default. If there is no title, the notice appears. I need to dig deeper and reproduce the bug later on.

@OpenCartAddons
Copy link
Contributor

This was caused due to a missing double quote in the banner_form.tpl. The fix is here #3399

Regards,
Joel.

@bhavin30
Copy link
Author

thank you :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants