Posts:
7,364
Registered:
31-Jan-2006
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
18-Nov-2018 16:39
in response to: AndreWolff
|
|
|
And I could have a button.inc file in the root directory and in some folders with a different contents of buttons.inc
OK, then: <ja:include page="${imageDirectory}/buttons.inc" />
It is better that this is corrected once in the jAlbum program instead that it should be corrected in all skins using .inc-files.
I agree that if you give it a relative path, like just buttons.inc, it should look in the current image directory, then in the root of the skin directory, then finally in the common includes directory. But if you code it explicitly, telling it to look only in the current image directory, you never have to rely on that cascade of searches.
|
|
|
Posts:
1,748
Registered:
14-Dec-2007
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
18-Nov-2018 19:58
in response to: jGromit
|
|
|
I agree that if you give it a relative path, like just buttons.inc, it should look in the current image directory, then in the root of the skin directory, then finally in the common includes directory.
That is not happening in version 17 and it should not happen.
Once again:
A new version should be compatibel with the previous version.
Discussion closed.
|
|
|
Posts:
7,364
Registered:
31-Jan-2006
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
18-Nov-2018 21:36
in response to: AndreWolff
|
|
|
I was agreeing with you, but just showing you how to avoid running into the problem entirely.
WTF is wrong with you?
|
|
|
Posts:
3,564
Registered:
18-Oct-2002
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
19-Nov-2018 19:32
in response to: jGromit
|
|
|
I was agreeing with you, but just showing you how to avoid running into the problem entirely.
WTF is wrong with you?
Thanks for spotting this change in behavior André! Let's call it a bug as it goes against the specification. I now see what caused it and will try to fix it.
Yes. The two of you are actually in agreeance. This is a typical case of "I don't hear what you say, I hear what my perception of you says. You have a history of heated arguing and that made André defensive and wanting to defend a standpoint both of you, and me, actually share  . This is so common. Guess why politicians can't ever agree with each other even though they eventually execute the same politics? :-D
|
|
|
Posts:
7,364
Registered:
31-Jan-2006
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
19-Nov-2018 20:11
in response to: davidekholm
|
|
|
Using a relative path for an include file is a bit like using the ${title} variable - you're never quite sure what you're going to get, because there's a cascade of possibilities. I'm guilty of it, as well, but I get away with it because I'm always going after something in the skin directory, and it always has a name that's not likely to cause a collision. It would be very odd, indeed, to find includes/objVars.inc in an image directory, or in the jAlbum program directory.
But upon reflection, the safer thing to do is always to use an absolute link, like ${skindirectory}/includes/objVars.inc. No surprises that way.
|
|
|
Posts:
3,105
Registered:
4-Aug-2006
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
19-Nov-2018 22:20
in response to: davidekholm
|
|
|
Not sure if it is because Groovy 3 is in beta, but import groovy.swing.SwingBuilder
fails. I was having a quick look at a simple interface for external tools, rather than a full-blown skin GUI.
|
|
|
Posts:
3,564
Registered:
18-Oct-2002
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
20-Nov-2018 12:14
in response to: RobM
|
|
|
|
Not sure if it is because Groovy 3 is in beta, but import groovy.swing.SwingBuilder
fails. I was having a quick look at a simple interface for external tools, rather than a full-blown skin GUI.
I've included the minimal amount of libraries needed to get Groovy running withing jAlbum. For that import to work too, we have to include a third library (attached). That library is supposed to allow declarative construction of Swing based UIs. If you fall in love with it, let me know!
The attached library might need more depending libraries. Get all you need via http://groovy-lang.org/download.html (see the Alpha-3 binaries download, then put all you want under jAlbum's lib folder)
|
|
|
Posts:
3,564
Registered:
18-Oct-2002
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
20-Nov-2018 14:02
in response to: davidekholm
|
|
|
|
|
Posts:
1,748
Registered:
14-Dec-2007
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
20-Nov-2018 17:25
in response to: davidekholm
|
|
|
Yes, I can confirm that inc-files are again processed as in the last released version, as you can see here.
Thanks!
|
|
|
Posts:
3,105
Registered:
4-Aug-2006
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
20-Nov-2018 21:48
in response to: davidekholm
|
|
|
I've included the minimal amount of libraries needed to get Groovy running withing jAlbum. For that import to work too, we have to include a third library (attached). That library is supposed to allow declarative construction of Swing based UIs. If you fall in love with it, let me know!
Well, I'm liking a lot of it, though it is different it doesn't seem that you need to start again from scratch - so picking it up should be easier - famous last words  . What I'm liking is that it is easier to read what you have written, for example I like not having to use the get methods count = 0
AlbumObject[] cao = JAlbumContext.getInstance().getExplorer().explorer.getSelectedAlbumObjects();
for (AlbumObject ao : cao) {
println count++
println ao.title
println ao.included
println ao.category
println ao.vars.cameraMake
}
The attached library might need more depending libraries. Get all you need via http://groovy-lang.org/download.html (see the Alpha-3 binaries download, then put all you want under jAlbum's lib folder)
I've downloaded all of the Alpha-3 binaries, so if I need anything else I can try adding those resources.
Of course, it needs more users to take an interest in it to make it worthwhile - unless Beanshell suddenly gives up the ghost and another language becomes mandatory. For now I'm happy having a play with it.
|
|
|
Posts:
3,105
Registered:
4-Aug-2006
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
20-Nov-2018 23:05
in response to: davidekholm
|
|
|
I just tried to update using my Beta update tool and it reported no update available, so to be sure I downloaded the above file and it too shows as Beta 4, not 5. Is it just a version error or is it the wrong file?
|
|
|
Posts:
7,364
Registered:
31-Jan-2006
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
20-Nov-2018 23:21
in response to: RobM
|
|
|
I just tried to update using my Beta update tool and it reported no update available, so to be sure I downloaded the above file and it too shows as Beta 4, not 5. Is it just a version error or is it the wrong file?
The splash screen says b4, but the "About" page reveals that it's b5.
|
|
|
Posts:
3,105
Registered:
4-Aug-2006
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
20-Nov-2018 23:25
in response to: jGromit
|
|
|
|
I just tried to update using my Beta update tool and it reported no update available, so to be sure I downloaded the above file and it too shows as Beta 4, not 5. Is it just a version error or is it the wrong file?
The splash screen says b4, but the "About" page reveals that it's b5.
The 'About' on My Mac is showing b4, see attached
|
|
|
Posts:
7,364
Registered:
31-Jan-2006
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
20-Nov-2018 23:29
in response to: RobM
|
|
|
|
Well, you'll just have to get a real PC. Actually, for the price of a Mac, you could get two pretty powerful PC's. 
|
|
|
Posts:
3,105
Registered:
4-Aug-2006
|
|
|
Re: jAlbum 17.1 beta for testing
Posted:
21-Nov-2018 00:43
in response to: jGromit
|
|
|
Well, you'll just have to get a real PC. Actually, for the price of a Mac, you could get two pretty powerful PC's. 
That sounds awful, so I cleared caches and downloaded again, and now it shows 17.1b5 in the about screen.
|
|
|
|
Legend
|
|
Forum admins
|
|
Helpful Answer
|
|
Correct Answer
|
|