Smalltalk › Frameworks & Tools › Seaside › Seaside General

WAUrl

_‹ Previous Topic Next Topic _›

Classic

List

Threaded

5 messages Options

rshiple

WAUrl

Should I have to subclass WAUrl to avoid WAHtmlDocument in a printOn: ?

>>printOn: aStream
| document codec |

codec := [ self requestContext codec ]
on: WARequestContextNotFound
do: [ :e | WANullCodec new ].
document := WACurlDocument on: aStream codec: codec.
self encodeOn: document usingHtmlEntities: false

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

Lukas Renggli

Re: WAUrl

This seems to be wrong. Please file a bug report.

Lukas

2009/9/11 Robert Shiplett <[hidden email]>

Should I have to subclass WAUrl to avoid WAHtmlDocument in a printOn: ?

>>printOn: aStream
| document codec |

codec := [ self requestContext codec ]
on: WARequestContextNotFound
do: [ :e | WANullCodec new ].
document := WACurlDocument on: aStream codec: codec.
self encodeOn: document usingHtmlEntities: false

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

--
Lukas Renggli
http://www.lukas-renggli.ch

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

Julian Fitzell-2

Re: WAUrl

Agreed. This is ugly and has been bothering me for a while. Not sure of the solution off the top of my head and might have to wait for improvements to encoding in 3.1 but do, please, file a bug report for it.

Julian

On Sat, Sep 12, 2009 at 10:13 AM, Lukas Renggli <[hidden email]> wrote:

This seems to be wrong. Please file a bug report.

Lukas

2009/9/11 Robert Shiplett <[hidden email]>

Should I have to subclass WAUrl to avoid WAHtmlDocument in a printOn: ?

>>printOn: aStream
| document codec |

codec := [ self requestContext codec ]
on: WARequestContextNotFound
do: [ :e | WANullCodec new ].
document := WACurlDocument on: aStream codec: codec.
self encodeOn: document usingHtmlEntities: false

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

--
Lukas Renggli
http://www.lukas-renggli.ch

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

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

Philippe Marschall

Re: WAUrl

2009/9/12 Julian Fitzell <[hidden email]>
>
> Agreed. This is ugly and has been bothering me for a while. Not sure of the solution off the top of my head and might have to wait for improvements to encoding in 3.1 but do, please, file a bug report for it.

I would rather say we ask the document whether we should use HTML
entities. Not nice, not oo, but works.

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

Philippe Marschall

Re: WAUrl

2009/9/12, Philippe Marschall <[hidden email]>:
> 2009/9/12 Julian Fitzell <[hidden email]>
>>
>> Agreed. This is ugly and has been bothering me for a while. Not sure of
>> the solution off the top of my head and might have to wait for
>> improvements to encoding in 3.1 but do, please, file a bug report for it.
>
> I would rather say we ask the document whether we should use HTML
> entities. Not nice, not oo, but works.

After staring low enough at the code I think I found an acceptable solution:
http://code.google.com/p/seaside/issues/detail?id=461

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