[Seaside] SVG Mime type

[Esteban Maringolo]

Hi,

I have the same version as the one in Pharo.

isBinary
"answers whether the contents of a document of the receiving mime type
are binary"

self main = 'text'  ifTrue: [ ^ false ].
self main = 'application'
ifTrue: [ "application/json is text"
self sub = 'json'
ifTrue: [ ^ false ] ]
ifFalse: [ ^ true ].
GRPlatform
subStringsIn: self sub
splitBy: $+
do: [ :each |
"application/x-javascript and application/xml are text"
(#('x-javascript' 'xml') includes: each)
ifTrue: [ ^ false ] ].
^ true

ps: The formatting is scrambled, Gmail's fault. :-/


Esteban A. Maringolo

On Wed, Jul 1, 2020 at 10:28 AM Annick Fron <list at afceurope.com> wrote:
>
> Hi Esteban,
>
>         Could you give us your isBinary method ?
>
>         Best regards
>
>         Annick
>
> > Le 1 juil. 2020 à 15:26, Esteban Maringolo <emaringolo at gmail.com> a écrit :
> >
> > As a side note... according to somebody from Cincom (I don't remember
> > his name) the latest VisualWorks (9.x) comes with Seaside 3.3.
> >
> > And I must say that the way callbacks and other things work within VW
> > is really solid, the only drawback is that the Seaside adapter is
> > based on their "old" HTTP server stack, and not the new one (SiouX)
> > which has HTTP/2 and many other improvements.
> >
> > The only problem I see is that the whole porting, maintenance is
> > opaque to the rest of the community. As it seems to be with other
> > frameworks as well.
> >
> > But if you need to know anything about Seaside in VW, I'll be happy to
> > help you as much as I can. I've been using for the past years, and
> > have some overrides and updated things I use regularly
> > (WAMimeType>>#isBinary is one of the overridden methods I have).
> >
> > Regards,
> >
> > Esteban A. Maringolo
> >
> > On Wed, Jul 1, 2020 at 7:29 AM Johan Brichau <johan at inceptive.be> wrote:
> >>
> >> Annick,
> >>
> >> It’s my understanding that Seaside in Visualworks is no longer updated with more recent versions of Seaside.
> >> Although, it seems that in a current Seaside, the following also returns true:
> >>
> >> (WAMimeType main: 'image' sub:'svg+xml’) isBinary
> >>
> >> I submitted an issue https://github.com/SeasideSt/Seaside/issues/1207 to track and correct.
> >> There are a couple more similar issues, so they will be treated as one ;)
> >>
> >> Johan
> >>
> >> On 1 Jul 2020, at 12:19, Annick Fron <list at afceurope.com> wrote:
> >>
> >> Hi,
> >> Thank you for the pointer.
> >> I have discovered that in the Seaside-Visualworks package in WAResponse>>document:, there is a test to check for binaries or text. And svg mime type is considered as binary (from WAMimeType isBinary method), so the svgContent method should be a string with the svg with asByteArray at the end.
> >> Best regards
> >>
> >> Annick
> >>
> >>
> >> Le 30 juin 2020 à 18:58, Paul DeBruicker <pdebruic at gmail.com> a écrit :
> >>
> >> That sounds like it should work.
> >>
> >> Where does the error occur?  In the browser or whatever smalltalk version
> >> you're using?
> >>
> >> Can you inspect the response in the browser and check that the correct mime
> >> type is set?
> >>
> >> Does your SVG file have a starting tag like the one shown here:
> >>
> >> https://developer.mozilla.org/en-US/docs/Web/SVG/Tutorial/Getting_Started
> >>
> >>
> >> Annick wrote
> >>
> >> Hi,
> >>
> >> I have tried to use the WAFileMetadataLibrary for importing svg files,
> >> everything works ok locally, but when publishing to my web site, I get an
> >> error « The mimetype of image/svg+xml » does not match the content.
> >>
> >> Anybody has an idea about this ?
> >>
> >> Best regards
> >>
> >> Annick
> >> _______________________________________________
> >> seaside mailing list
> >>
> >>
> >> seaside at .squeakfoundation
> >>
> >>
> >> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> >>
> >>
> >>
> >>
> >>
> >>
> >> --
> >> Sent from: http://forum.world.st/Seaside-General-f86180.html
> >> _______________________________________________
> >> seaside mailing list
> >> seaside at lists.squeakfoundation.org
> >> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> >>
> >>
> >> _______________________________________________
> >> seaside mailing list
> >> seaside at lists.squeakfoundation.org
> >> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> >>
> >>
> >> _______________________________________________
> >> seaside mailing list
> >> seaside at lists.squeakfoundation.org
> >> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> > _______________________________________________
> > seaside mailing list
> > seaside at lists.squeakfoundation.org
> > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>
> _______________________________________________
> seaside mailing list
> seaside at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside