Posts:
4,307
Registered:
18-Oct-2002
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 13:38
in response to: JeffTucker
|
|
|
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. |
|
|
Posts:
8,237
Registered:
31-Jan-2006
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 13:45
in response to: davidekholm
|
|
|
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. |
|
|
Posts:
4,307
Registered:
18-Oct-2002
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 14:00
in response to: JeffTucker
|
|
|
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 |
|
|
Posts:
4,307
Registered:
18-Oct-2002
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 14:06
in response to: JeffTucker
|
|
|
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.
|
|
|
Posts:
4,307
Registered:
18-Oct-2002
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 14:07
in response to: JeffTucker
|
|
|
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. |
|
|
Posts:
4,307
Registered:
18-Oct-2002
|
|
|
|
Re: Make changes log
Posted:
13 May 20, 14:11
in response to: karlmistelberger
|
|
|
Exception in thread "AWT-EventQueue-0" java.lang.UnsupportedOperationException: Taskbar API is not supported on the current platform
Now fixed. |
|
|
Posts:
4,307
Registered:
18-Oct-2002
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 14:12
in response to: AndreWolff
|
|
|
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. |
|
|
Posts:
8,237
Registered:
31-Jan-2006
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 14:20
in response to: davidekholm
|
|
|
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.... |
|
|
Posts:
4,307
Registered:
18-Oct-2002
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 14:31
in response to: JeffTucker
|
|
|
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
|
|
|
Posts:
8,237
Registered:
31-Jan-2006
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 14:49
in response to: davidekholm
|
|
|
...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.  |
|
|
Posts:
8,237
Registered:
31-Jan-2006
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 14:54
in response to: JeffTucker
|
|
|
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. |
|
|
Posts:
497
Registered:
27-Sep-2003
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 16:43
in response to: davidekholm
|
|
|
- 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. |
|
|
Posts:
4,307
Registered:
18-Oct-2002
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 17:25
in response to: ctwist
|
|
|
- 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. |
|
|
Posts:
8,237
Registered:
31-Jan-2006
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 17:25
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.
|
|
|
Posts:
4,307
Registered:
18-Oct-2002
|
|
|
|
Re: jAlbum 20.1 beta for testing
Posted:
13 May 20, 17:26
in response to: JeffTucker
|
|
|
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
|
|
