Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Class comments of WAKomEncoded and WAKomEncoded39 are lacking #51

Closed
GoogleCodeExporter opened this issue Mar 25, 2015 · 2 comments
Closed

Comments

@GoogleCodeExporter
Copy link

The class comments of WAKomEncoded and WAKomEncoded39 do not mentions the 
dependency on 
the Kom version

Original issue reported on code.google.com by philippe...@gmail.com on 8 Apr 2008 at 4:38

@GoogleCodeExporter
Copy link
Author

In Seaside2.8a1-pmm.540 the class comments read:

WAKomEncoded:
 provide an adapter between Seaside and the Comanche web server with response/request conversion from 
UTF-8 to WideString and back in Squeak 3.8 or later.

see WAKom

Use me if you have either Squeak 3.8 and an old version of KomHttpServer or 
Squeak 3.9 and a new version of 
KomHttpServer. Squeak 3.8 and an new version of KomHttpServer are likely to 
cause problems. If you use 
Squeak 3.9 or later and an earlier version of KomHttpServer se WAKomEncoded39.

definition of a new version of Kom:
- SqueakMap: KomHttpServer 7.0.5 or later
- Monticello: KomHttpServer KomHttpServer-RJT.23 or later, especially 
KomHttpServer-gc.27 or later

images in which you should use WAKomEncoded
- the Seaside One-Click image
- a recent squeak-web image from Damien

WAKomEncoded39:
I have the same resposibility like WAKomEncoded except that I'm for Squeak 3.9 
and old version of 
KomHttpServer. See class comment of WAKomEncoded for more details. In general I 
should not be used 
anymore.

Original comment by philippe...@gmail.com on 8 Apr 2008 at 5:01

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

fixed dude

Original comment by philippe...@gmail.com on 13 Apr 2008 at 6:22

  • Changed state: Fixed
  • Added labels: ****
  • Removed labels: ****

dalehenrich added a commit to dalehenrich/Seaside that referenced this issue Sep 7, 2015
dalehenrich added a commit to dalehenrich/Seaside that referenced this issue Sep 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant