search

OWASP-BLT / BLT-Flutter

The official OWASP BLT App repository/ Heist 'em bugs!
https://owasp.org/www-project-bug-logging-tool/
BSD 3-Clause "New" or "Revised" License
22 stars 29 forks source link

Fixing range error when no screenShot for issue is there #357

Closed Dante291 closed 6 months ago

Dante291 commented 6 months ago

fixes #356

Basically we were getting range error because we were trying to access first element of screenshots list of an issue which was null so I added a null check such that if list is null we show a default issue image.

Screenshot_1708799047

Dante291 commented 6 months ago

@DonnieBLT Let me know if this default image (bug image which matches BLT theme) for an issue with no screenshots is ok?

DonnieBLT commented 6 months ago

There should be no issues without screenshots. That's a bug that we've been having that needs to be fixed so I think what we can do instead is just don't show the issue. How about that?

Dante291 commented 6 months ago

There should be no issues without screenshots. That's a bug that we've been having that needs to be fixed so I think what we can do instead is just don't show the issue. How about that?

Alright then we need a check to see if there is any screenshot for a issue, if no then dont show that issue also should i add a functionnality in the app that user shouldn't be able to create an issue if there is no screenshot attached?

DonnieBLT commented 6 months ago

That would be good. Thanks so much.