Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Gwenview Gwenview
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 4
    • Issues 4
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 13
    • Merge requests 13
  • 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
  • GraphicsGraphics
  • GwenviewGwenview
  • Merge requests
  • !180

Don't export slideshow data on MPRIS when no slideshow is happening, fix base64 path generation for dbus

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Joshua Goins requested to merge work/redstrate/fix-mpris into master Jan 18, 2023
  • Overview 21
  • Commits 2
  • Pipelines 10
  • Changes 4

This is an alternative MR to !179 (closed), which fixes a couple of bugs related to MPRIS instead of outright removing it.

  • Slideshow data is exported, even when there was no slideshow happening. There's no bug reported about this, but there are a couple of off-hand reports about how weird this feature is. For example, open an image in fullscreen mode, and then back out - Gwenview still shows in MPRIS. Now it only shows when a slideshow is actually happening. Tested on KDE connect too, and it's still possible to remotely start slideshows there too!
  • Fixing BUG: 402970 because paths like /tmp/a3o is one of the few paths that generate Base64 strings with + in them, which is actually not a valid DBus path.
Edited Jan 18, 2023 by Joshua Goins
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: work/redstrate/fix-mpris