hoffmannrobert created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. hoffmannrobert requested review of this revision.
REVISION SUMMARY If your filter list has two matching filters for an extension, KFileWidget will return the name of the first filter when calling QFileDialog::selectedNameFilter(), even if the second filter was selected in the dialog. This is a follow-up for https://phabricator.kde.org/D21249 TEST PLAN 1. Compile and run the attached program to the bug report 407819 2. Make sure that "auto extension" checkbox is enabled 3. Select the last file filter (DocBook (.xml)) via dropdown list 4. Click "Save" 5. Check program output Result without this patch: The output is "Word 2003 XML (.xml) (*.xml)" Result with this patch applied: The output should be "DocBook (.xml) (*.xml)", since this was selected in the file dialog. Fixes bug 407819 REPOSITORY R135 Integration for Qt applications in Plasma BRANCH fix_selectedNameFilter REVISION DETAIL https://phabricator.kde.org/D21959 AFFECTED FILES autotests/kfiledialog_unittest.cpp src/platformtheme/kdeplatformfiledialogbase_p.h src/platformtheme/kdeplatformfiledialoghelper.cpp src/platformtheme/kdeplatformfiledialoghelper.h src/platformtheme/kdirselectdialog.cpp src/platformtheme/kdirselectdialog_p.h To: hoffmannrobert Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart