'Legacy banner' issue
Bug #1859602 reported by
Ovidiu Suciu
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Software Center Agent |
Fix Released
|
Undecided
|
Hasan Ammar |
Bug Description
We have this issue https:/
The name match on "banner.[png|jpeg]" is too aggressive and should only be matched against if the file is of type "screenshot".
Currently is seems to be matching against type "banner".
Changed in software-center-agent: | |
assignee: | nobody → Hasan Ammar (hasanammar) |
status: | New → In Progress |
To post a comment you must log in.
Suspect code:
``` /src/devportal/ forms.py: 2004 legacy_ banner( uploaded_ file.name, image):
dashboard:
# XXX cprov 2019-03-14: quick-and-dirty support for the
# GnomeSoftware (legacy) banner & banner-icon as specially
# named screenshots. Once clients get proper support for
# banner, this 'if' and the helper method can be
# removed along with the corresponding tests.
if not self._is_
```
_is_legacy_banner tests for LEGACY_BANNER_NAMES = ('banner.png', 'banner.jpg')