Re: [Glass] Unicode16 DNU seasideMimeDocument

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: [Glass] Unicode16 DNU seasideMimeDocument

JupiterJones
Good point :)

Maybe those methods need to move to the nearest common superclass - CharacterCollection.

I’m copying this to the seaside list - perhaps someone there could offer more educated advice.

Cheers,

J

On 26 Mar 2021, at 5:05 am, bruno buzzi brassesco via Glass <[hidden email]> wrote:

Hi,

But the receiver object is anUnicode16 not aString.

regards,

bruno

On 24/3/2021 19:13, Jupiter Jones wrote:
Hey Bruno,

String-#seasideMimeDocument is in the package Seaside-Core.

Does loading that package resolve the issue? Is this in a stone with all seaside loaded, or were you just testing REST by loading the REST group?

Cheers,

J

On 25 Mar 2021, at 8:21 am, bruno buzzi brassesco via Glass <[hidden email]> wrote:

Hi,

This is just a report.

Doing some tests on a REST layer i came across this error.

Unicode16  DNU seasideMimeDocument (GS 3.5.4) -Seaside was loaded from https://github.com/SeasideSt/Seaside/-

Not sure in which github repository report it... Or send a PR because is an easy fix:

Unicode16
seasideMimeDocument
	^ WAMimeDocument on: self mimeType: WAMimeType textPlain

regards,
bruno
_______________________________________________
Glass mailing list
[hidden email]
https://lists.gemtalksystems.com/mailman/listinfo/glass

_______________________________________________
Glass mailing list
[hidden email]
https://lists.gemtalksystems.com/mailman/listinfo/glass


_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: [Glass] Unicode16 DNU seasideMimeDocument

BrunoBB

Hi,

https://github.com/SeasideSt/Seaside/pull/1247

It add CharacterCollection>>seasideMimeDocument to Seaside-GemstoneCore but does not remove String>>seasideMimeDocument which is in Seaside-Core.

reagards,
bruno

On 26/3/2021 11:34, Johan Brichau via Glass wrote:

Hi,

Methods that should be on CharacterCollection in GemStone and not on String… that is indeed something we often encounter in the GemStone port :-)
So, might very well be a bug… can you submit it on the GitHub? a PR would be great as well.

Thanks!
Johan

On 25 Mar 2021, at 20:51, Jupiter Jones via Glass <[hidden email]> wrote:

Good point :)

Maybe those methods need to move to the nearest common superclass - CharacterCollection.

I’m copying this to the seaside list - perhaps someone there could offer more educated advice.

Cheers,

J

On 26 Mar 2021, at 5:05 am, bruno buzzi brassesco via Glass <[hidden email]> wrote:

Hi,

But the receiver object is anUnicode16 not aString.

regards,

bruno

On 24/3/2021 19:13, Jupiter Jones wrote:
Hey Bruno,

String-#seasideMimeDocument is in the package Seaside-Core.

Does loading that package resolve the issue? Is this in a stone with all seaside loaded, or were you just testing REST by loading the REST group?

Cheers,

J

On 25 Mar 2021, at 8:21 am, bruno buzzi brassesco via Glass <[hidden email]> wrote:

Hi,

This is just a report.

Doing some tests on a REST layer i came across this error.

Unicode16  DNU seasideMimeDocument (GS 3.5.4) -Seaside was loaded from https://github.com/SeasideSt/Seaside/-

Not sure in which github repository report it... Or send a PR because is an easy fix:

Unicode16
seasideMimeDocument
	^ WAMimeDocument on: self mimeType: WAMimeType textPlain

regards,
bruno
_______________________________________________
Glass mailing list
[hidden email]
https://lists.gemtalksystems.com/mailman/listinfo/glass

_______________________________________________
Glass mailing list
[hidden email]
https://lists.gemtalksystems.com/mailman/listinfo/glass

_______________________________________________
Glass mailing list
[hidden email]
https://lists.gemtalksystems.com/mailman/listinfo/glass


_______________________________________________
Glass mailing list
[hidden email]
https://lists.gemtalksystems.com/mailman/listinfo/glass

_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside