Skip to content

Conversation

piotreke12
Copy link
Contributor

This feature adds possibility to search only in specified upload while currently it is possible to do it in all available uploads

Closes: #1280

Description

Search option for all available uplads only is not very useful when there is more than 3-5 uploads user has access to.
This feature adds possibility to choose in which upload user wants to search in.
List od available uploads for user is dynamically loaded from DB.

Changes

  • Added option-field on template src/www/ui/search-helper.php filled with all uploads user has access to (plus option to choose to search in All uploads)
  • Added method for src/lib/php/Dao/UploadDao.php to retrieve all uploads
  • modified src/www/ui/search-helper.php#GetResults to filter search results only for selected upload

How to test

Search results should be shown only for Upload selected from dropdown menu.
For --All uploads -- results from all uploads should be shown (as previously, before this feature)

Copy link
Member

@GMishx GMishx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few suggestions.

@GMishx
Copy link
Member

GMishx commented Oct 1, 2020

There are few linting failures

image

@piotreke12 piotreke12 force-pushed the feat/orange-opensource/1280/restrict-search-scope branch from 2f6083f to a9b069f Compare October 1, 2020 08:58
@piotreke12
Copy link
Contributor Author

There are few linting failures

Yes, sorry - forgot to push updated version - done

@piotreke12 piotreke12 force-pushed the feat/orange-opensource/1280/restrict-search-scope branch from a9b069f to 763f9b9 Compare October 1, 2020 09:23
@piotreke12 piotreke12 force-pushed the feat/orange-opensource/1280/restrict-search-scope branch 2 times, most recently from 6e7d5a4 to 6a865a8 Compare October 21, 2020 07:30
@piotreke12 piotreke12 force-pushed the feat/orange-opensource/1280/restrict-search-scope branch from 6a865a8 to 5e4fae2 Compare October 26, 2020 15:50
Copy link
Member

@GMishx GMishx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes looks good.

Copy link
Member

@GMishx GMishx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested, working fine.

@GMishx GMishx added ready and removed needs test labels Oct 28, 2020
@GMishx GMishx added this to the 3.10.0 milestone Nov 26, 2020
@GMishx GMishx merged commit 81bbe53 into fossology:master Dec 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve Search option to limit the results to one project

2 participants