Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Plasma Desktop Plasma Desktop
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 10
    • Issues 10
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 47
    • Merge requests 47
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Plasma
  • Plasma DesktopPlasma Desktop
  • Merge requests
  • !776

kcms/baloo: Fix state checking and button behaviour

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Yerrey Dev requested to merge yerrey/plasma-desktop-2:indexerstatefix2 into master Jan 02, 2022
  • Overview 3
  • Commits 2
  • Pipelines 8
  • Changes 1

Currently currentlyIndexing function always returns true as monitor.IndexerState isn't a thing. This causes the button to always show "Pause Indexer".

This MR fixes the issue of the IndexerState state enum being checked incorrectly in the switch statement, and also the QQC2.Button text being show incorrectly no matter whether the indexing is suspended or not. The reason currentlyIndexing function cannot be used in the button text is because it also has values of Idle, LowPowerIdle, Unavailable, Startup that should show "Pause Indexer" as the indexer is still enabled even if it has nothing to do at the moment.

Edited Jan 03, 2022 by Yerrey Dev
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: indexerstatefix2