Forum OpenACS Q&A: Re: Leading space in APD pages

Collapse
4: Re: Leading space in APD pages (response to 3)
Posted by Andrei Popov on 12/02/03 12:20 PM

I think it is put there in acs-templating somewhere, not sure where though. I guess it would be worth to look at compiled template and see if there's anything that might cause this.

I had a problem with this only because I was doing some post-processing using xsltproc on the generated page. Ended up doing a (quick'n dirty) sed -i 's/^ <!DOCTYPE/<!DOCTYPE/' on the wget'ed file...

Collapse
5: Re: Leading space in APD pages (response to 4)
Posted by Jeff Davis on 12/02/03 12:37 PM
I fixed this a while back on HEAD.  I will see if I can track down the commit.
Collapse
6: Re: Leading space in APD pages (response to 5)
Posted by Jeff Davis on 12/02/03 12:44 PM
Here it is: 
cvs diff -u -r1.10 -r 1.11 acs-integration-procs.tcl
Index: acs-integration-procs.tcl
===================================================================
RCS file: /cvsroot/openacs-4/packages/acs-templating/tcl/acs-integration-procs.tcl,v
retrieving revision 1.10
retrieving revision 1.11
diff -u -r1.10 -r1.11
--- acs-integration-procs.tcl   1 Oct 2003 14:51:41 -0000       1.10
+++ acs-integration-procs.tcl   10 Oct 2003 19:15:04 -0000      1.11
@@ -5,7 +5,7 @@
 # Copyright (C) 1999-2000 ArsDigita Corporation
 # Authors: Christian Brechbuehler <christian@arsdigita.com

-# $Id: acs-integration-procs.tcl,v 1.10 2003/10/01 14:51:41 lars Exp $
+# $Id: acs-integration-procs.tcl,v 1.11 2003/10/10 19:15:04 jeffd Exp $

 # This is free software distributed under the terms of the GNU Public
 # License.  Full text of the license is available from the GNU Project:
@@ -158,7 +158,7 @@

         set mime_type [template::get_mime_type]
         set header_preamble [template::get_mime_header_preamble $mime_type]
-       doc_return 200 $mime_type "$header_preamble $parsed_template"
+       doc_return 200 $mime_type "$header_preamble$parsed_template"
     } else {
         db_release_unused_handles
     }