Bug in WebGridColumn?

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

Bug in WebGridColumn?

Martin Polák
Hi Janko,

like me or not, but i found something bad happen in newest beta1.9 :). If you try to login as admin and click on Profile, than on Sessions, debugger is open in Pharo. So I looked around and found problem with WebGridColumn, so I update the method autoConvert:

WebGridColumn>>autoConvert: anObject
        (anObject class == Date and: [self parent hasShortDates])
                ifTrue: [^anObject shorterPrintSloString].
        (anObject class == SpDate) "Sport portable date"
                ifTrue: [^anObject shorterPrintSloString].
        (anObject class == SpTimestamp) "Sport portable timestamp"
                ifTrue: [^anObject printString].
        (anObject = 0 and: [self parent hasNoZeros]) ifTrue: [^''].
        ^WebFormElement autoConvertToString: anObject

after this update, Sessions seems to operate normally.

Martin
_______________________________________________
Aida mailing list
[hidden email]
http://lists.aidaweb.si/mailman/listinfo/aida
Reply | Threaded
Open this post in threaded view
|

Re: Bug in WebGridColumn?

Janko Mivšek
Ahoj Martin,

S, Martin Polák piše:

> like me or not, but i found something bad happen in newest beta1.9 :). If you try to login as admin and click on Profile, than on Sessions, debugger is open in Pharo. So I looked around and found problem with WebGridColumn, so I update the method autoConvert:
>
> WebGridColumn>>autoConvert: anObject
> (anObject class == Date and: [self parent hasShortDates])
> ifTrue: [^anObject shorterPrintSloString].
> (anObject class == SpDate) "Sport portable date"
> ifTrue: [^anObject shorterPrintSloString].
> (anObject class == SpTimestamp) "Sport portable timestamp"
> ifTrue: [^anObject printString].
> (anObject = 0 and: [self parent hasNoZeros]) ifTrue: [^''].
> ^WebFormElement autoConvertToString: anObject
>
> after this update, Sessions seems to operate normally.

Your patch integrated. Funny i s that on VW this works, where is the
difference with Pharo .. Well, it doesn't matter for now, let we jsut
have in mind for the future.

Thanks again!
Janko


--
Janko Mivšek
Aida/Web
Smalltalk Web Application Server
http://www.aidaweb.si
_______________________________________________
Aida mailing list
[hidden email]
http://lists.aidaweb.si/mailman/listinfo/aida