Forum OpenACS Development: Noquoting development and testing server up
The noquote dev and testing server is up and running and the changes that were made so far look quite promising. I'm now inviting the community to help out a fair bit with testing. The server can be found here : http://noquote.cph02.collaboraid.net:20000.
If you find pages with dubious html, please setup a bug at http://openacs.org/bugtracker/openacs/ or drop me a line.
Oh, this one seems to get them right, too. Why's that?
My test server accepts ISO-8859-1 as well after I followed the hints at the bottom of this thread:
A good suggestion: It's declarative and not hidden somewhere in the code. And as Jim says, it also doesn't bother the template designers.
The @varname;noquote@ was probably conceived for two reasons:
- It provided for a relatively easy migration.
- It was implemented under deadline pressure.
- you need to noquote every variable (with html code) that you pass on to an included adp
- you need to noquote a property you pass on to a master template
(IIRC Peter was working on the project that originally "noquoted" ACS - maybe he knows more)
There's a few more rather nasty things with the current noquote patch e.g.:
However I don't have an idea yet how a more radical change would work in real life. Noquoting a system with a more proper approach would take a lot of time and most certainly lead to two different codebases that are hard to bring together.
If we do it step by step - live with a solution that works but will be phased out eventually - then I think it is workable.
Also some templating system guru has to be found to implement the code for the proper solution. I'm not really savvy about the templating system itself, but I'd be happy to give some directions.
Excellent points. We should keep going with the work you have already done. I see no reason to not allow specifying noquote in the adp file.
Perhaps for another version we can improve it even more, allowing an option of specifying in the -properties of the tcl file.
- you would have to add a similar thing to multirows since multirows are not further declared in the -properties list
- you will only get the variable content quoted or not, so if you want to use the quoted and the unquoted variable content in the same adp, then you're lost....
In addition to that, i want to point out that some action on that issue of quoting should be taken as soon as possible. I've worked with the patched version for 2 years already (yeah, i know..... we should have asked the community sooner to adopt this change) and this proved to be a valuable and stable addition.
I've uploaded the noquote work I did so far to file-storage. Find it here: http://openacs.org/storage/?folder_id=73696.
It has a bunch of sed oneliners to automatically noquote as many templates as possible and a patch that does the necessary changes to parse-procs.tcl and other tcl or adp pages I touched.