Created attachment 50562 [details] Screenshot with tab "Status unknown" displaying all suggestions To reproduce: - Test on Top of Bug 16322 for better visualization - Create some suggestions - Go to Home > Acquisitions > Suggestions management - Edit some suggestions and give them different status, e.g. accepted, rejected, pending. - Verify that they appear in the tabs as appropriate - Edit one suggestion, set "Mark selected as" to --Choose a status-- - Result: New tab "Status unknown" containing all suggestions (expected: tab contains only suggestion with "Unknown status") See screenshot attached.
Created attachment 51153 [details] [review] Bug 16325: Add a test for SearchSuggestions when searching for STATUS=''
Created attachment 51154 [details] [review] Bug 16325: Do not return all suggestions if search for STATUS='' This is a quick and dirty way to fix a bad bug in a messy area. The "unknown status" tab in the suggestions table display all the suggestions. It should only display suggestions with a STATUS='' Test plan: - Create some suggestions - Go to Home > Acquisitions > Suggestions management - Edit some suggestions and give them different status, e.g. accepted, rejected, pending. - Verify that they appear in the tabs as appropriate - Edit one suggestion, set "Mark selected as" to --Choose a status-- => Without this patch: New tab "Status unknown" containing all suggestions => With this patch: tab contains only suggestions with "Unknown status"
Created attachment 51159 [details] [review] Bug 16325: Add a test for SearchSuggestions when searching for STATUS='' Test fails as expected without second patch and passes OK with second patch. Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 51160 [details] [review] Bug 16325: Do not return all suggestions if search for STATUS='' This is a quick and dirty way to fix a bad bug in a messy area. The "unknown status" tab in the suggestions table display all the suggestions. It should only display suggestions with a STATUS='' Test plan: - Create some suggestions - Go to Home > Acquisitions > Suggestions management - Edit some suggestions and give them different status, e.g. accepted, rejected, pending. - Verify that they appear in the tabs as appropriate - Edit one suggestion, set "Mark selected as" to --Choose a status-- => Without this patch: New tab "Status unknown" containing all suggestions => With this patch: tab contains only suggestions with "Unknown status" Works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 51259 [details] [review] [PASSED QA] Bug 16325: Add a test for SearchSuggestions when searching for STATUS='' Test fails as expected without second patch and passes OK with second patch. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 51260 [details] [review] [PASSED QA] Bug 16325: Do not return all suggestions if search for STATUS='' This is a quick and dirty way to fix a bad bug in a messy area. The "unknown status" tab in the suggestions table display all the suggestions. It should only display suggestions with a STATUS='' Test plan: - Create some suggestions - Go to Home > Acquisitions > Suggestions management - Edit some suggestions and give them different status, e.g. accepted, rejected, pending. - Verify that they appear in the tabs as appropriate - Edit one suggestion, set "Mark selected as" to --Choose a status-- => Without this patch: New tab "Status unknown" containing all suggestions => With this patch: tab contains only suggestions with "Unknown status" Works as expected. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to Master - Should be in the May 2016 Release. Thanks!
Patch pushed to 3.22.x, will be in 3.22.7