Adding new Banner

[curryman]

Getting this error when adding new banner

******* = admin dir

[23-Apr-2025 20:15:45 UTC] Request URI: /test/*******/index.php?cmd=banner_manager&action=add, IP address: **.***.132.213
--> PHP Fatal error: Uncaught TypeError: Cannot access offset of type string on string in /home/spiceshopuk/public_html/test/includes/classes/upload.php:140
Stack trace:
#0 /home/spiceshopuk/public_html/test/*******/banner_manager.php(134): upload->save()
#1 /home/spiceshopuk/public_html/test/*******/index.php(16): require('/home/spiceshop...')
#2 {main}
thrown in /home/spiceshopuk/public_html/test/includes/classes/upload.php on line 140.


Website shows 500 error

Any ideas?

[wmorris]

Getting this error when adding new banner

******* = admin dir

[23-Apr-2025 20:15:45 UTC] Request URI: /test/*******/index.php?cmd=banner_manager&action=add, IP address: **.***.132.213
--> PHP Fatal error: Uncaught TypeError: Cannot access offset of type string on string in /home/spiceshopuk/public_html/test/includes/classes/upload.php:140
Stack trace:
#0 /home/spiceshopuk/public_html/test/*******/banner_manager.php(134): upload->save()
#1 /home/spiceshopuk/public_html/test/*******/index.php(16): require('/home/spiceshop...')
#2 {main}
thrown in /home/spiceshopuk/public_html/test/includes/classes/upload.php on line 140.


Website shows 500 error

Any ideas?

First of all, what zen cart version are you using and what PHP version?

[curryman]

Hi,

Zencart 2.1

PHP 8.0

[simon1066]

Yes, this was a bug and occurs when saving a banner without an image. If you replace your copy of admin/banner_manager.php with the latest from the GitHub repo you should be good to go.

https://github.com/zencart/zencart/b...er_manager.php

[curryman]

Great worked, many thanks