[Seaside-dev] Problems encountered in the file-in of Seaside packages in Dolphin

[Philippe Marschall]

2009/9/16 GallegO <fxgallego at gmail.com>:
> Hi list:
>
> As promised earlier I will comment on some problems that I found in
> the file-in / port of Seaside to Dolphin.
> I'm talking *only* about packages that should be considered as dialect neutral.
>
> Begining...
> Package RSS-Core
> How to handle local days, month and year names when printing. I had to
> change the encodeOn: method to print the names in English and not in
> Spanish. I'm not sure how Pharo, VW and others proceeds with the
> Locale date names but this can be cause of errors in systems with
> other languages than English. This can / should be moved to a
> WAXxxPlatform?
> This is the current version for Dolphin:
> RRRfc822DateTime>>encodeOn:
> encodeOn: aDocument
>
>        Locale evaluate: [
>                aDocument
>                        nextPutAll: (dateAndTime dayOfWeekName first: 3);
>                        nextPutAll: ', ';
>                        nextPutAll: (dateAndTime dayOfMonth printPaddedWith: $0 to: 2);
>                        nextPut: Character space;
>                        nextPutAll: (dateAndTime monthName first: 3);
>                        nextPut: Character space;
>                        nextPutAll: dateAndTime year seasideString;
>                        nextPut: Character space;
>                        nextPutAll: (dateAndTime hour24 printPaddedWith: $0 to: 2);
>                        nextPut: $:;
>                        nextPutAll: (dateAndTime minute printPaddedWith: $0 to: 2);
>                        nextPut: $:;
>                        nextPutAll: (dateAndTime second printPaddedWith: $0 to: 2);
>                        nextPut: Character space;
>                        nextPutAll: timezoneString]
>                usingLocale: Locale englishUS

Can you open a bug for this? It probably will have to be platform
specific in this case.

> In the Packages:
>  Scriptaculous-Components
>  Scriptaculous-Tests-Components
>  Scriptaculous-Tests-Core
>
> there are a lot of methods sending  #asString and I think that should
> send #seasideString
>
> --------------------------------------------------------------------------
>
> Can be replaced in SUWindow>>setJSOrigin:
> the statement
> stringArray := aString subStrings: (Array with: $,).
> by
> stringArray := aString subStrings: (String with: $,).?????
>
> This is because Dolphin is not ANSI on subStrings: implementation.
> In Pharo is safe. If this can not be changed I should override the
> Dolphin original with one provided in Seaside-Dolphin-Platform
>
> ----------------------------------------------------------------------------
>
> SUClassBrowser>>categories
> SUClassBrowser>>renderSourceOn:
> SUClassBrowser>>selectors
> Should rely on a platform specific code to get source, categories,
> selectors, etc ????
>
> ------------------------------------------------------------------------------
>
> SwazooSite>>sslPort:
> SwazooSite>>host:ip:port:sslPort:
> both methods contains a reference to SSLSiteIdentifier that is
> undefined on Pharo 1.0 beta

That's a Swazoo bug, please report against Swazoo.

Cheers
Philippe