[Seaside] SVG Mime type

[Johan Brichau]

Hi,

In latest Seaside on GitHub, the error is not present.

(WAMimeType main: 'image' sub:'svg+xml') isBinary
=> false

I must have made a mistake while quickly checking when I responded earlier.

Anything with ‘xml’ as the sub part, is considered non-binary.
See https://github.com/SeasideSt/Seaside/blob/master/repository/Seaside-Core.package/WAMimeType.class/instance/isBinary.st <https://github.com/SeasideSt/Seaside/blob/master/repository/Seaside-Core.package/WAMimeType.class/instance/isBinary.st>

So, my guess is that VW version is not correct?

Johan

> On 1 Jul 2020, at 15:28, 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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/seaside/attachments/20200703/db154496/attachment.html>