Thread Locked This thread is locked - replies are not allowed.



Permlink Replies: 110 - Pages: 8 [ Previous | 1 2 3 4 5 6 7 8 | Next ] - Last Post: 14 May 20, 15:17 Last Post By: JeffTucker Threads: [ Previous | Next ]
davidekholm

Posts: 3,662
Registered: 18-Oct-2002
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 13:38   in response to: JeffTucker in response to: JeffTucker
jGromit wrote:
davidekholm wrote:
These errors mostly mean that the user has left a stale album folder behind....

That's not the situation he's describing. Let's say you have a MyDog project, and a MyCat project, and you're using a skin that creates a res directory in the image directory, to store a logo file, for example. Now you want to create a MyPets project, with your dog and cat as folders, so you drag in the original separate projects, each of which contains a res folder.

Rare case, but it could happen.

ETA: I believe I've detected David trying to take a shortcut to ignoring a leftover album directory, but this approach backfires in some strange ways. ;)


True. I don't want to introduce too much network disk IO when building albums over network drives by checking multiple contained files and folders just for this little convenience. Anyway. you're right it shouldn't backfire in a situation like that. I'm now ignoring folders containing an "album.rss" file. Not 100% tight either, but this will make the risk of inserting old album folders into an album pretty small.
JeffTucker

Posts: 7,360
Registered: 31-Jan-2006
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 13:45   in response to: davidekholm in response to: davidekholm
davidekholm wrote:
True. I don't want to introduce too much network disk IO when building albums over network drives by checking multiple contained files and folders just for this little convenience. Anyway. you're right it shouldn't backfire in a situation like that. I'm now ignoring folders containing an "album.rss" file. Not 100% tight either, but this will make the risk of inserting old album folders into an album pretty small.

Hmmm.... I no longer include RSS in any of my albums, because it was causing 404's on the server when dealing with an album of albums (a long story, and a separate issue, so let's not wade into that here). How about ignoring any folder that has an index.html in it (of course, picking up the current index and extension name, rather than hard-coding it). Even tighter, I would think.
davidekholm

Posts: 3,662
Registered: 18-Oct-2002
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 14:00   in response to: JeffTucker in response to: JeffTucker
jGromit wrote:
Consistent.

ETA: It happens only if the default skin does not support full site pages. So, if I launch, create a new project, switch to, say, Neptune, the choice is grayed out. If I switch back to Minimal, it's OK, too. In fact, if I start a new project, then just choose Minimal again, it works properly. :)

Edited by: jGromit on 13-May-2020 07:38


Now fixed in the latest beta
davidekholm

Posts: 3,662
Registered: 18-Oct-2002
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 14:06   in response to: JeffTucker in response to: JeffTucker
jGromit wrote:

Hmmm.... I no longer include RSS in any of my albums, because it was causing 404's on the server when dealing with an album of albums (a long story, and a separate issue, so let's not wade into that here). How about ignoring any folder that has an index.html in it (of course, picking up the current index and extension name, rather than hard-coding it). Even tighter, I would think.

Ok, changed to detecting the index file instead.
davidekholm

Posts: 3,662
Registered: 18-Oct-2002
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 14:07   in response to: JeffTucker in response to: JeffTucker
jGromit wrote:
davidekholm wrote:
Moves physical album folder if the location of the album folder is edited under Settings - General

Not working in Win10. OK in macOS.


I haven't implemented support for moving across drive letters. Also, the move (which is a rename operation) only succeeds if there is no such target folder already.
davidekholm

Posts: 3,662
Registered: 18-Oct-2002
Re: Make changes log
Posted: 13 May 20, 14:11   in response to: karlmistelberger in response to: karlmistelberger
karlmistelberger wrote:
Exception in thread "AWT-EventQueue-0" java.lang.UnsupportedOperationException: Taskbar API is not supported on the current platform

Now fixed.
davidekholm

Posts: 3,662
Registered: 18-Oct-2002
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 14:12   in response to: AndreWolff in response to: AndreWolff
AndreWolff wrote:
This beta version crashed with big albums like my Algarve album:
An unexpected error occurred. Please see detailed description below:
 
java.lang.OutOfMemoryError: Java heap space
	at java.base/java.util.Arrays.copyOf(Unknown Source)
	at java.base/java.lang.AbstractStringBuilder.ensureCa
where I changed already the init file:
Virtual Machine Parameters=-Xms64M -Xmx6400M -Djalbum.installation.dir="C:\Program Files\jAlbum\" -Dmy.pictures.path="K:\Users\Andre\Afbeeldingen\" -Djava.net.preferIPv4Stack=true -Djna.nosys=true -XX:+UseParallelGC
This problem is absent in released version 20.0.6

I need to see a longer stack trace.
JeffTucker

Posts: 7,360
Registered: 31-Jan-2006
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 14:20   in response to: davidekholm in response to: davidekholm
davidekholm wrote:
jGromit wrote:
davidekholm wrote:
Moves physical album folder if the location of the album folder is edited under Settings - General

Not working in Win10. OK in macOS.

I haven't implemented support for moving across drive letters. Also, the move (which is a rename operation) only succeeds if there is no such target folder already.


I can confirm that it works if the target doesn't already exist.

But I think that might be a problem. If I want to change the output directory, I don't want to have to manually type in a new location. I want to click the little folder icon, and navigate to the new location. Typically, a user would have a "whole site" tree, on which he wants to hang the albums. So, the target directories would already exist.

And there's at least a fair chance that the target would be on another drive letter.

Not sure what to suggest. A "move" with forced overwrite? But what if the target isn't empty? I can see a lot of thrashing about to avoid problems....
davidekholm

Posts: 3,662
Registered: 18-Oct-2002
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 14:31   in response to: JeffTucker in response to: JeffTucker
jGromit wrote:

I can confirm that it works if the target doesn't already exist.

But I think that might be a problem. If I want to change the output directory, I don't want to have to manually type in a new location. I want to click the little folder icon, and navigate to the new location. Typically, a user would have a "whole site" tree, on which he wants to hang the albums. So, the target directories would already exist.


Ok, I now also perform moves to empty target directories, but as far as handling different drive letters? Really? We're into corner cases now I think ;-)
JeffTucker

Posts: 7,360
Registered: 31-Jan-2006
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 14:49   in response to: davidekholm in response to: davidekholm
davidekholm wrote:
...as far as handling different drive letters? Really? We're into corner cases now I think ;-)

Probably not. From what I've seen in these forums, people who keep a complete site mirror on their PC's often keep it on a separate drive. That's not a bad idea, since it helps keep things organized. But maybe we can trust them to take care of things themselves.

Oh, who am I kidding? These are precisely the users who will demand extensive hand-holding.
JeffTucker

Posts: 7,360
Registered: 31-Jan-2006
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 14:54   in response to: JeffTucker in response to: JeffTucker
jGromit wrote:
ETA: It happens only if the default skin does not support full site pages. So, if I launch, create a new project, switch to, say, Neptune, the choice is grayed out. If I switch back to Minimal, it's OK, too. In fact, if I start a new project, then just choose Minimal again, it works properly. :)

Given how "fringe-y" this case is, I wouldn't sweat this one, unless it's a very easy fix. After all, adding a page template doesn't do any real harm, since the skin will ignore it anyway. In fact, my skins will explicitly exclude them (with a warning dialog), so the user is in no doubt about why it's not in the finished album.
ctwist

Posts: 636
Registered: 27-Sep-2003
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 16:43   in response to: davidekholm in response to: davidekholm
davidekholm wrote:
  • Moves physical album folder if the location of the album folder is edited under Settings - General
Could be dangerous.

The user may have accidentally entered the album location of a diffferent project. He realizes this and corrects it. He doesn't want the other project's album to be moved, and then overwritten when the current project is made.

Suggestion: Only move the album folder if the current project has already generated an album in the previous album folder location.
davidekholm

Posts: 3,662
Registered: 18-Oct-2002
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 17:25   in response to: ctwist in response to: ctwist
ctwist wrote:
davidekholm wrote:
  • Moves physical album folder if the location of the album folder is edited under Settings - General
Could be dangerous.

The user may have accidentally entered the album location of a diffferent project. He realizes this and corrects it. He doesn't want the other project's album to be moved, and then overwritten when the current project is made.

Suggestion: Only move the album folder if the current project has already generated an album in the previous album folder location.


Isn't the current scheme safe enough? We only move the physical album folder if the target folder is either missing or empty.
JeffTucker

Posts: 7,360
Registered: 31-Jan-2006
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 17:25   in response to: ctwist in response to: ctwist
Possible in theory, but is that really any worse than, for example, accidentally hitting the minus key and excluding a folder, which deletes the output? Worst case, you'd have to open the other project and Make Album again.
davidekholm

Posts: 3,662
Registered: 18-Oct-2002
Re: jAlbum 20.1 beta for testing
Posted: 13 May 20, 17:26   in response to: JeffTucker in response to: JeffTucker
jGromit wrote:

Given how "fringe-y" this case is, I wouldn't sweat this one, unless it's a very easy fix. After all, adding a page template doesn't do any real harm, since the skin will ignore it anyway. In fact, my skins will explicitly exclude them (with a warning dialog), so the user is in no doubt about why it's not in the finished album.

The current beta should have this one fixed now.
Legend
Forum admins
Helpful Answer
Correct Answer

Point your RSS reader here for a feed of the latest messages in all forums