Forum OpenACS Q&A: I know this is simple, but I'm flumoxed --

Seems I've made some small error in my "default-master.adp" file. List of file:
html
head
title @title@ /title
LINK href="/mobetta.css" type=text/css rel=StyleSheet /head
body
@header_stuff@
multiple name=attribute @attribute.key@="@attribute.value@" /multiple

slave

hr
address a href="mailto:@signatory@" @signatory@ /a /address
@ds_link@ br
div align="center" © pcs-sc.com, all rights reserved! Please e-
mail for permission to reprint 
  or use in any way. /div
/body
/html
*Brackets removed to display code...

All of my templated pages are now printing "bgcolor="white" text="black" " at the top. I know these are set in the default- master.tcl, but everything was working fine until one of my edits an hour ago. Now I'm afraid to muck around too much in fear I'll mess it up even more.

Maybe someone could forward a copy of the default-master adp and tcl file for comparison.

Thanks,

Collapse
Posted by Jon Griffin on
Because bgcolor needs to be in body, not after it.
Collapse
Posted by Gilbert Price on
Jon,

Thanks for the quick response, but if you'll notice above, the multiple name=attribute @attribute.key@="@attribute.value@" /multiple line does come after the body tag and not before it. That is one of the reasons I'm perplexed. I know this is an easy fix, obviously I did something without thinking and broke it...

Collapse
Posted by Janine Ohmer on
Gilbert,
<p>
The original goes like this:
<p>
<pre>
html
head
title@title@/title
@header_stuff@
/head
body multiple name=attribute@attribute.key@="@attribute.value@"/multiple

slave
</pre>
The significant differences are that @header_stuff@ is within the HEAD block, and the multiple is within the BODY tag.  I'm not sure which is causing your problem but they both look significant to me.

Collapse
Posted by Jon Griffin on
Without your tags in there I can't tell, it still looks wrong.

It should be <body @header_stuff@ multiple name=attribute @attribute.key@="@attribute.value@" /multiple >

If it is like that then I am sorry, but you really need to post with the < and >

Collapse
Posted by Jon Griffin on
I had a typo, but Janine's is correct. The multiples aren't in the body.
Collapse
Posted by Gilbert Price on
Thanks for the help,
<p>
This works:
<pre>
html (brackets)
head (brackets)
title (brackets)
<b>@title@</b>
/title (brackets)
LINK href="/mobetta.css" type=text/css rel=StyleSheet (brackets)
/head (brackets)
body (brackets)
<b>@header_stuff@</b>
multiple (brackets)
<b>name=attribute @attribute.key@="@attribute.value@"</b>
/multiple (brackets)

slave (brackets)
  ...

</pre>

<b>Bold items have no brackets...</b>
<p>
My error was including this tag <multiple name=attribute> rather than <multiple> then the rest of the line. The @header_stuff@ actually goes after the <body> tag, if I put before the body tag there was no change to my error. Once I fixed the <multiple> tag the errant text disappeared, and I guess is now actually applied to the page.
<p>
Thanks again for the help, you got me thinking instead of staring at tags...