elvisangelaccio created this revision.
elvisangelaccio added a reviewer: Plasma.
elvisangelaccio added a subscriber: Plasma.
elvisangelaccio set the repository for this revision to rPLASMAINTEGRATION 
Integration for Qt applications in Plasma.
elvisangelaccio added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  For some reason, the `kde2QtFilter()` function returns an empty string when 
the filedialog has more than one mimetype set (through `setMimeTypeFilters()`) 
and the user selects a specific filter (among the default filters provided by 
those mimetypes).
  
  This patch just adds a simple fallback when that function is about to return 
an empty string.

TEST PLAN
  - build/tests/qfiledialogtest --mimeFilter application/json --mimeFilter 
application/zip
  - Select the json filter
  - Select a json file
  - Check the "selected name filter" in the debug output. Without patch -> 
empty string, with the patch: `JSON document (*.json)`

REPOSITORY
  rPLASMAINTEGRATION Integration for Qt applications in Plasma

REVISION DETAIL
  https://phabricator.kde.org/D1813

AFFECTED FILES
  src/platformtheme/kdeplatformfiledialoghelper.cpp
  tests/qfiledialogtest.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: elvisangelaccio, #plasma
Cc: plasma-devel, #plasma, sebas
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to