LauncherEditorDialog - can't output new file

Bug #458342 reported by Michal Hruby
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Desktop Agnostic Library for GLib-based Projects
Fix Released
Medium
Mark Lee

Bug Description

As the merge is closed, I'll report here:
>> + if (this._output.exists () && !this._output.is_writable ())

> Not needed, and gives an unintended side effect. What if the output path is /root/foo and the user is not a superuser?

If we want to create new desktop file (by passing newly created instance of VFSFile as 2nd parameter of LauncherEditorDialog constructor), lda just displays the "Save as" dialog, bothering the user, who will save it to incorrect location anyway.

Related branches

Mark Lee (malept)
Changed in libdesktop-agnostic:
assignee: nobody → Mark Lee (malept)
importance: Undecided → Medium
milestone: none → 0.3.9
status: New → Triaged
Revision history for this message
Mark Lee (malept) wrote :

Fix committed in revision 380.
The algorithm is now:
if output file exists {
    if output file is not writable, show "Save As" dialog
} else {
    if parent folder is not writable, show "Save As" dialog
}

Changed in libdesktop-agnostic:
status: Triaged → Fix Committed
Mark Lee (malept)
Changed in libdesktop-agnostic:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.