This small patch allows to set and empty type to the extended options, which will set the Any Type option (index 0).
It doesn't make any change in current code, since this case is not called, but it helps simplifying follow-up patches.
Details
Details
- Reviewers
elvisangelaccio - Group Reviewers
Dolphin - Commits
- R318:adfc098aa747: fix(search): Allow to set empty type
No behavior changes with current code.
Calling setFacetsType(QString()) actually sets the first option (Any Type)
Diff Detail
Diff Detail
- Repository
- R318 Dolphin
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
To elaborate, in general we can push to the stable branch (release/19.12 currently) as long as we don't introduce new strings or new features and until the branch gets "frozen" for the final release tagging (5th december this time: https://community.kde.org/Schedules/Applications/19.12_Release_Schedule).
Big refactoring commits should still go to master though.