Defect #521
Change introduced in rev 76 prevents Smart client working in 6.0.6
0%
Description
I believe globals.js line 62:
if (Packages.com.servoy.j2db.ClientVersion.getReleaseNumber()>= 1400 /*Servoy 6.1 or higher*/) {
is not working in a real Smart client (perhaps some ClassLoading issue or security related issue).
This works in developer but not in a real Smart client...
This makes the whole globals.DIALOGS unusable (it is null).
Reverting to rev 75 fixes it.
I propose to use a simple parsing of application.getVersion() instead of relying on non public API! ;)
Files
History
Updated by Robert Ivens almost 12 years ago
- Status changed from New to Rejected
This is already fixed by revision 77. Which is the latest revision in trunk at the time of this writing :)
Updated by Patrick Talbot almost 12 years ago
Then it's the change of rev 77 which is causing a problem...
All I know is that I could see on a server of a client of mine that this wasn't working with the latest version, so I reverted to rev 75 that I know was working... The result was a JavaScript error on globals.DIALOGS.showErrorMessage(), saying that showErrorMessage() couldn't be called on null...
I deduced it was the call to Packages.com.servoy.j2db.ClientVersion.getReleaseNumber();
but perhaps its the call to new Continuation(); that is not working???
All I'm saying is that it needs to be tested in a real client....
Updated by Robert Ivens almost 12 years ago
- Status changed from Rejected to Reopened
Hmm...well continuations shouldn't be used with smart client anyway, it's webclient only.
Will investigate.
Updated by Patrick Talbot almost 12 years ago
- File DIALOGS-null.png DIALOGS-null.png added
Well in that case, there is a need for some conditional code, but I just tested it in Smart client with rev77 and got the same error (working fine in Web client and Debug client from Developer, and Web client from server, but not working in Smart client from server).
Updated by Robert Ivens almost 12 years ago
- Status changed from Reopened to Feedback
The code is full of conditional code. I did spot one discrepancy though. The terminator variable was declared with a continuation. I made it conditional.
Can you check revision 78 and see if this fixes it for you?
Updated by Patrick Talbot almost 12 years ago
- Status changed from Feedback to Closed
Working now with rev >= 78. Thanks!