pydio / pydio-core

Pydio 8 official repository
https://pydio.com
GNU Affero General Public License v3.0
867 stars 289 forks source link

Added option for Lucene Search to search only the current folder and subfolders #1329

Closed bures closed 7 years ago

bures commented 7 years ago

I would like to use Pydio as for digital asset management. There a few features missing that I believe are needed. The ability to search only the current folder and its subfolders is one of them. I tried to implement this for index.lucene. There is an option in workspace features where one can enable the search in current folder and subfolders (as opposed to the global workspace-wide search). The solution uses an additional field in the Lucene index, thus reindexing is needed after the update.

cdujeu commented 7 years ago

Hi Tomas, Welcome on board! That's indeed a very interesting feature, i'll have a deep look at it. I would probably try to add the option (folder / not folder) directly on the UI side, to let the user choose, what do you think? I can do the changes after the merge. Charles

bures commented 7 years ago

Hello Charles. Happy to be onboard. I think the option of folder/not folder directly in the search UI would be better (along with the ability to set the default in workspace features settings). Thanks. T.

cdujeu commented 7 years ago

hey thomas, just coming back on that now... don't worry about the conflicts/merge, i'll put your suggestions manually in the new branch. Still, i'm not sure to understand why you would need to introduce this new node_dir field and not use directly the node_path field? As we index it as a "keyword" , it's not broken into pieces, and looking for "currentDir*" should give the expected results, no? [edit1] Ok you are using a "text" instead of a "keyword" [edit2] But we do use "node_path" for such a wildcard query in the getIndexedDocumentsChildren(), so that should work... charles

bures commented 7 years ago

Hello Charles,

yes, I originally wanted to use the existing field. But in the end this is what I converged to eventually. The keyword type didn't work for me.

T.

On 4/29/2017 18:20, Charles du Jeu wrote:

ok forget it, i just get my answer : we must in fact use a "text" instead of a keyword to perform the wildcard query ;-)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/pydio/pydio-core/pull/1329#issuecomment-298178577, or mute the thread https://github.com/notifications/unsubscribe-auth/ABpi18LLdMyCmN9OD4RAqP7YzyF_ae7kks5r02NGgaJpZM4MLdrx.

-- Tomas Bures, Ph.D. Associate Professor Department of Distributed and Dependable Systems Charles University Malostranske nam.25 11800 Prague 1, Czech Republic http://d3s.mff.cuni.cz Phone: (+420) 95155 4236 Fax: (+420) 95155 4323