[comixed] Re: Code testing

  • From: "Darryl L. Pierce" <mcpierce@xxxxxxxxx>
  • To: comixed@xxxxxxxxxxxxx
  • Date: Fri, 28 Aug 2020 08:42:31 -0400

Cool, so the new issue I opened for 0.7 should take care of fixing that
problem with special characters. And I'll look at what needs to be done to
ensure that, on failure, the original filename and comic entry are restored.

On Fri, Aug 28, 2020 at 8:38 AM bareheiny <dmarc-noreply@xxxxxxxxxxxxx>
wrote:

I left everything blank - mostly because I’m not sure of the format for
populating it.

The special character was in the series name, definitely not in the
existing file name.  In the case of a failure, can the original file be
left?  The log noting the failure will point the user in the right
direction - rather than them wondering where the comic went, and why
there’s an “old” version.


On 28/08/2020, at 23:32, Darryl L. Pierce <mcpierce@xxxxxxxxx> wrote:


What were the renaming rules and the target directory passed in? If the
former is left blank then the original filename should be used, and if the
latter is blank then it's going to move things to the root of the drive.

On Fri, Aug 28, 2020 at 5:18 AM bareheiny <dmarc-noreply@xxxxxxxxxxxxx>
wrote:

Also not a fan of the renaming / moving.  All my comics ended up at the
root of the drive they’re on.  Not too bad given I’m using a removable, but
not ideal if they were on my c drive.

On 28/08/2020, at 21:08, bareheiny <bareheiny@xxxxxxxxx> wrote:


Will give it a go a bit later.

Tried the consolidation - cx crashed eventually, but first it failed on
one comic.

If a comic series has a special character (such as :), the file isn’t
moved....but it is renamed with an “.old” extension (or similar).


On 28/08/2020, at 00:05, Darryl L. Pierce <mcpierce@xxxxxxxxx> wrote:


It should be showing comics that are missing. When the backends loads an
update to send to the frontend, it checks to see if the physical file is
present and sets the flag accordingly.

(checks code)

I see the issue. The request that returns the update was mistakenly
filtering out the missing flag on each comic. It should be fixed now on
develop.

On Thu, Aug 27, 2020 at 7:31 AM bareheiny <dmarc-noreply@xxxxxxxxxxxxx>
wrote:

Another question, should the page accessed via the  “Missing” menu item
show comics that aren’t in the expected location - or is it an incomplete
feature?  If incomplete, maybe consider disabling the menu item for now.


On 27/08/2020, at 09:03, bareheiny <bareheiny@xxxxxxxxx> wrote:


I’ll give the conversion another test, and document any odd behaviour.

Not a fan of the library organising though.  I have a structure that, in
some cases, deviates from the standard Publisher \ Imprint \ Series \
Volume structure.

For example I’m grouping some comics by a primary character - so all
Venom titles are going into Venom \, with no sub folders.

My preference would be for CX to allow users to chose whether or not
files are reorganised.



On 26/08/2020, at 23:45, Darryl L. Pierce <mcpierce@xxxxxxxxx> wrote:


On Tue, Aug 25, 2020 at 6:27 PM bareheiny <dmarc-noreply@xxxxxxxxxxxxx>
wrote:

I’ve been using 0.7 for a bit....not many issues, most I’ve found
relate to incomplete features.

Will use it a bit more going forward though.

Questions:

 The library consolidation - can the file renaming be disabled by the
user, and is there guides for building the naming structure?


If no renaming rule is passed to the backend, then the filename for the
comic isn't changed, and the file's copied into the top of the target
directory.


File converting sometimes doesn’t remove the original item - bug or
incomplete feature?


Definitely a bug.


Also, should conversion also update reading lists?  So if I convert a
comic that’s on a reading list, the reading list entry would be updated to
point to the converted version of the comic.


It should, yes.

--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." -
Gord Downie



--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." -
Gord Downie



--
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." - Gord
Downie



-- 
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." - Gord
Downie

Other related posts: