Find programmers and grapic design experts at ScriptLance.com
<? DEW-CODE.COM  
Dew-Code
Welcome, Guest
Please Login or Register.    Lost Password?
Exactly what causes this code? (1 viewing) (1) Guest
Go to bottom Post Reply Favoured: 0
TOPIC: Exactly what causes this code?
#124
jnasta (User)
Senior Boarder
Posts: 70
graphgraph
User Offline Click here to see the profile of this user
Gender: Male John Nasta Photography Location: Brooklyn, NY
Exactly what causes this code? 6 Months, 4 Weeks ago Karma: 1  
in the main directory page and other category pages...

<tr><td class="regularText"></td></tr> </td>
</tr>
</table>
</td>
</tr>

</table>
<!-- End New Table -->

<!-- Start New Table -->


Clearly this:

</td></tr> </td>

is wrong.

Thanks
 
Report to moderator   Logged Logged  
  The administrator has disabled public write access.
#130
Dewed (Moderator)
Moderator
Posts: 229
graph
User Online Now Click here to see the profile of this user
Dew-Code.com
Re:Exactly what causes this code? 6 Months, 4 Weeks ago Karma: 7  
Keen eye !

This appears to come from /include/review_inc.php around lines 30 - 38

A flaw in my logic I suppose, assuming a featured site would have at least one review. Short term fix.. ensure your featured sites have at least one review.

Long term... hrm...

Code:

$html = $table2 . "<tr><td class=\"regularText\">"; if(isset($av_rows["average"])){ $html .= " - " . $review_3 . "<font class=\"regularBoldText\">"; $html .= $av_rows["average"] . "</font>"; } $html .= "</td></tr>";
Scratching my head, I'm clearly missing something. Possibly.. the queries that rely on $SiteID being set... if for whatever reason its not set, the queries would fail, resulting in.. well wonkiness for lack of a better term. I'll have to revisit this, but if you determine anything further, please enlighten me
 
Report to moderator   Logged Logged  
 
Nothing to it but to Dew it !
Dew-Code.com
  The administrator has disabled public write access.
#133
jnasta (User)
Senior Boarder
Posts: 70
graphgraph
User Offline Click here to see the profile of this user
Gender: Male John Nasta Photography Location: Brooklyn, NY
Re:Exactly what causes this code? 6 Months, 4 Weeks ago Karma: 1  
I'm wondering if that should read

</td></tr></table>

There seems to be a missing </table> tag on that page somewhere.

If you copy & paste the entire output source code of the page and one by one remove the nested tables from within each other, you eventually end up w/ no closing tags for the main table.

This seems to make sense b/c if I add the </table> tag there then everything after the blockquote and between <!-- Start New Table --> and <!-- End New Table --> comes out correctly.

If that is left as <tr><td> w/ no closing </table> tag the tables go wonky as you say.

John
 
Report to moderator   Logged Logged  
  The administrator has disabled public write access.
#137
fattymattybrewing (User)
Support Your Local Brewer
Expert Boarder
Posts: 105
graphgraph
User Offline Click here to see the profile of this user
Gender: Male Fatty Matty Brewing Location: Southwestern Wisconsin Birthdate: 1974-11-05
Re:Exactly what causes this code? 6 Months, 4 Weeks ago Karma: 3  
for sure I am lost...but actually the truth of this on my site...I have the reviews "not showing" or as what I like to call it - the reviews are turned off on my site, I wasn't sure if this was the correct way to disable it but I just commented out review code. Just an FYI that it can be done.
 
Report to moderator   Logged Logged  
 
Last Edit: 2008/05/09 12:07 By fattymattybrewing. Reason: explain statement
 
Lifes Too Short To Drink Crappy Beers
fattymattybrewing.com
  The administrator has disabled public write access.
#140
jnasta (User)
Senior Boarder
Posts: 70
graphgraph
User Offline Click here to see the profile of this user
Gender: Male John Nasta Photography Location: Brooklyn, NY
Re:Exactly what causes this code? 6 Months, 4 Weeks ago Karma: 1  
I still haven't figured it out either but there is definitely something wonky in the table setup there.

To make your site a fixed width and centered, it seems like the easiest thing to do is add another set of table tabs between the body tags in the directory file. At least by doing that it seems I've got everything contained.

Now if I could just make it actually work...

I'm still having pathing issues. Adding those lines to directory didn't help.
 
Report to moderator   Logged Logged  
  The administrator has disabled public write access.
Go to top Post Reply
Powered by FireBoardget the latest posts directly to your desktop

Newsflash

Sign up for PayPal and start accepting credit card payments instantly.
Copyright Dew-Code 2008