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

[Bug]: Error shown in category search (upload photo) when no input is provided #5872

Open
zihanpan opened this issue Oct 19, 2024 · 1 comment · May be fixed by #5884
Open

[Bug]: Error shown in category search (upload photo) when no input is provided #5872

zihanpan opened this issue Oct 19, 2024 · 1 comment · May be fixed by #5884
Assignees

Comments

@zihanpan
Copy link
Contributor

Summary

When uploading a photo, the step 3 is performing a category search. At the moment I get in that page, an error message ' no categories found' is incorrectly displayed even when the search input field is empty. Then the editText will not accept any input. This behavior is unexpected because no search should trigger an error message when no input is provided. The error message should only appear when a valid search term is entered and no results are found.

Steps to reproduce

  1. Click the '+' in the right bottom of main page.
  2. Choose the second button which means select an existing photo
  3. Choose a photo
  4. Finish the step1 and step2
  5. Enter the page of step 3, the error message will show immediately and disappear. Then no matter what you type in, the edit text wont accept it and won't search.

Expected behaviour

The system should not display an error message if user has just enter that page.

Actual behaviour

The error message show immediately and disappear. Then no matter what you type in, the edit text wont accept it and won't search.

Device name

Android Emulators

Android version

Android 14

Commons app version

master

Device logs

2024-10-19 02:11:18.611 8054-8054 AutofillManager fr.free.nrw.commons.beta D view not autofillable - not passing ime action check

Screen-shots

No response

Would you like to work on the issue?

Yes

@zihanpan zihanpan added the bug label Oct 19, 2024
@zihanpan
Copy link
Contributor Author

Screen_recording_20241019_025434_compressed.mp4

I found this bug when I work on #4895. I think it is because showError of onAttachView did not check if the term is not blank.
So if possible, could I work on this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment