Opened 12 years ago

Closed 11 years ago

Last modified 11 months ago

#8519 closed enhancement (fixed)

Expander should have option to create directories that don't exist.

Reported by: kallisti5 Owned by: korli
Priority: normal Milestone: R1
Component: Applications/Expander Version: R1/Development
Keywords: Cc: przemub@…
Blocked By: Blocking:
Platform: All

Description

  • Provide Expander a path to expand to that doesn't exist
  • Press extract.. Expander says "Destination folder doesn't exist"
  • A more useful feature may be to ask the user:
    "Destination folder doesn't exist, create?"
    <Yes> <No>
    

Attachments (3)

gci2012-expander-patch-8519.diff (2.1 KB ) - added by mr.Noisy 11 years ago.
A patch from Przemysław Buczkowski (GCI 2012)
commit-64eda3e.diff (1.9 KB ) - added by przemub 11 years ago.
A patch from Przemysław Buczkowski (GCI 2012) (corrected) (corrected)
patch.diff (1.6 KB ) - added by mr.Noisy 11 years ago.
Final patch with few codestyle fixes

Download all attachments as: .zip

Change History (14)

comment:1 by X512, 12 years ago

Some archives store only one folder and Expander handle it correctly, but some other store files and folders directly in archive. This archives should be extracted in separate folder(usually named as archive but without file extension).

Extra checkbox "Extract in separate folder" can be added.

comment:2 by scottmc, 12 years ago

Milestone: R1/alpha4Unscheduled

This is feature creep and should not be in the alpha4 milestone. Moving to unscheduled.

comment:3 by przemub, 11 years ago

Cc: przemub@… added

by mr.Noisy, 11 years ago

A patch from Przemysław Buczkowski (GCI 2012)

comment:4 by mr.Noisy, 11 years ago

patch: 01

comment:5 by diver, 11 years ago

Milestone: UnscheduledR1

comment:6 by siarzhuk, 11 years ago

Looks like this patch need some love - in case there are some nested nonexisted folders - data will be extracted in wrong folder. :-( For example: set the destination to /boot/home/desktop/uu/uu and start expanding - folders are created but files were extracted to Desktop instead of uu/uu folder.

comment:7 by przemub, 11 years ago

Indeed. :/ I'll find a workaround in a moment...

Version 0, edited 11 years ago by przemub (next)

by przemub, 11 years ago

Attachment: commit-64eda3e.diff added

A patch from Przemysław Buczkowski (GCI 2012) (corrected) (corrected)

comment:8 by przemub, 11 years ago

I found - it was necessary to create new BEntry after creating a directory path.

by mr.Noisy, 11 years ago

Attachment: patch.diff added

Final patch with few codestyle fixes

comment:9 by siarzhuk, 11 years ago

Resolution: fixed
Status: newclosed

Applied in hrev44946. thanks!

comment:10 by jeffreestar, 11 months ago

spam

Last edited 11 months ago by humdinger (previous) (diff)

comment:11 by katyperry, 11 months ago

spam

Last edited 11 months ago by madmax (previous) (diff)
Note: See TracTickets for help on using tickets.