bdevries (Bart De Vries)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Sunday

  • Clear sailing ahead.

User Details

User Since
Sep 18 2019, 7:17 PM (8 w, 1 d)
Availability
Available

Recent Activity

Fri, Nov 8

bdevries requested review of D25224: Solve a bug which can happen with multiple files with the same name..
Fri, Nov 8, 9:44 PM · Gwenview
bdevries added inline comments to D24875: Importer: convert all file operations to allow remote URLs..
Fri, Nov 8, 9:17 PM · Gwenview
bdevries updated the diff for D24875: Importer: convert all file operations to allow remote URLs..

A few additional changes based on Nate's comments.

Fri, Nov 8, 9:16 PM · Gwenview

Fri, Nov 1

bdevries requested review of D25111: Implement KMessageWidget to report import errors..
Fri, Nov 1, 10:50 PM · Gwenview

Thu, Oct 31

bdevries added a comment to D25016: Create additional subfolders if needed (e.g. when extra slashes in FileNameFormater)..

No, nothing bad will happen. The importer will try to create a subfolder if needed. If it somehow doesn't succeed, it will result in a FileUtils::RenameFailed.
In that case the file will simply be skipped altogether. It will remain untouched on the source device, and it will also not be deleted if the user tells the importer to delete the imported files. Deleting is only done for successfully imported files.
I'm already working on the other patch. Luckily, it seems like it might be relatively simple to implement some basic UI warnings for failed items.

Thu, Oct 31, 6:58 PM · Gwenview
bdevries added a comment to D25016: Create additional subfolders if needed (e.g. when extra slashes in FileNameFormater)..

For traceability's sake, I would separate the proper error handling into a separate patch/commit.
It's completely missing at the moment, so it seems strange to mix it in with a "new feature".

Thu, Oct 31, 3:41 PM · Gwenview

Wed, Oct 30

bdevries updated the diff for D25016: Create additional subfolders if needed (e.g. when extra slashes in FileNameFormater)..

Register as FileUtils::RenameFailed if the subfolder cannot be created.

Wed, Oct 30, 3:33 PM · Gwenview
bdevries added a comment to D25016: Create additional subfolders if needed (e.g. when extra slashes in FileNameFormater)..

You're absolutely right that there needs to be a GUI warning/report of some kind. However, the current importer does not have the infrastructure to handle this: it does nothing with failed imports (of any kind) except print a warning message to stdout. See e.g. the FIXME/warning on lines 208 and 255.

Wed, Oct 30, 10:40 AM · Gwenview

Mon, Oct 28

bdevries added a comment to D24881: After import, change default action to "keep" instead of "delete"..

I'll certainly consider it.
Let's see if I can find the time. :-)

Mon, Oct 28, 9:58 PM · Gwenview
bdevries added a reviewer for D25016: Create additional subfolders if needed (e.g. when extra slashes in FileNameFormater).: Gwenview.
Mon, Oct 28, 8:43 PM · Gwenview
bdevries requested review of D25016: Create additional subfolders if needed (e.g. when extra slashes in FileNameFormater)..
Mon, Oct 28, 8:40 PM · Gwenview
bdevries updated the diff for D24881: After import, change default action to "keep" instead of "delete"..

Swap keep and delete buttons. This makes keep the default action.

Mon, Oct 28, 8:24 PM · Gwenview

Thu, Oct 24

bdevries set the repository for D24875: Importer: convert all file operations to allow remote URLs. to R260 Gwenview.
Thu, Oct 24, 8:31 PM · Gwenview
bdevries updated the diff for D24875: Importer: convert all file operations to allow remote URLs..

Add unit tests and solve bug when mulitiple files of same name exist.

Thu, Oct 24, 8:28 PM · Gwenview
bdevries updated the diff for D24875: Importer: convert all file operations to allow remote URLs..

Use QScopedPointer for QIODevice pointers

Thu, Oct 24, 2:47 PM · Gwenview
bdevries added a comment to D24881: After import, change default action to "keep" instead of "delete"..

The purpose of this dialog box is to ask the user what should be done with the original files on the source device that the photos were imported from. I.e. deleting them on the source device, e.g. the camera or smartphone. Only the photos that were imported successfully are then deleted from the source device (so not the ones that failed or were not selected to be imported).
BTW, the dialog box pops up after the import itself has been done.

Thu, Oct 24, 9:28 AM · Gwenview

Wed, Oct 23

bdevries added a reviewer for D24881: After import, change default action to "keep" instead of "delete".: Gwenview.
Wed, Oct 23, 8:50 AM · Gwenview
bdevries requested review of D24881: After import, change default action to "keep" instead of "delete"..
Wed, Oct 23, 8:48 AM · Gwenview
bdevries added a comment to D24874: Fix temporary path in importer test unit.

Since this is my first ever contribution to KDE: is there anything that I still need to do on my side? Or will this commit be applied without further intervention from me?

Wed, Oct 23, 6:58 AM · Gwenview

Tue, Oct 22

bdevries abandoned D24876: Create additional subfolders if needed (e.g. when extra slashes in FileNameFormater)..
Tue, Oct 22, 9:43 PM · Gwenview
bdevries requested review of D24876: Create additional subfolders if needed (e.g. when extra slashes in FileNameFormater)..
Tue, Oct 22, 9:43 PM · Gwenview
bdevries added a comment to D24874: Fix temporary path in importer test unit.

Sure, my "git" email address is devriesb@gmail.com .

Tue, Oct 22, 8:52 PM · Gwenview
bdevries added a dependency for D24875: Importer: convert all file operations to allow remote URLs.: D24872: Replace mkdtemp by QTemporaryDir.
Tue, Oct 22, 8:48 PM · Gwenview
bdevries added a dependent revision for D24872: Replace mkdtemp by QTemporaryDir: D24875: Importer: convert all file operations to allow remote URLs..
Tue, Oct 22, 8:48 PM · Gwenview
bdevries requested review of D24875: Importer: convert all file operations to allow remote URLs..
Tue, Oct 22, 8:47 PM · Gwenview
bdevries requested review of D24874: Fix temporary path in importer test unit.
Tue, Oct 22, 8:33 PM · Gwenview
bdevries requested review of D24872: Replace mkdtemp by QTemporaryDir.
Tue, Oct 22, 8:26 PM · Gwenview