CloseTable Code Breaking Page

mauldus · **CloseTable Code Breaking Page -** 10-13-2004, 01:13 AM

I've altered my CloseTable function for a framed table but whenever I view the Content module as a user or guest, the pannel gives me the usuall Access Denied error but then CloseTable does not run after that and the code just stops, no more html after that. It fixes when I replace my tables.php file with the original one so it must be something in my CloseTable function code. Could someone with a fresh pair of eyes take a look at it. I can't find the hiccup. Thanks.

Code:
http://www.antifusion.net/tables.txt

Example:
http://ird.antifusion.net/modules.php?name=Content

Mighty_Y · 10-13-2004, 02:49 PM

i doubt this will work, but better try it anyway
remove

Code:

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

at the end of the closetable function

mauldus · 10-13-2004, 06:49 PM

It didn't work. I think the whole php instance is dieing. Something in that html seems to be killing the parser, if that makes sense.

Here is the original, working code for these functions for your comparison.
http://www.antifusion.net/working.txt

Still need suggestions.
Thanks.

chatserv · 10-14-2004, 12:46 AM

in footer.php find:

Code:

function foot() {
    global $prefix, $user_prefix, $db, $index, $user, $cookie, $storynum, $user, $cookie, $Default_Theme, $foot1, $foot2, $foot3, $foot4, $home, $module, $name;

change to:

Code:

function foot() {
    global $prefix, $user_prefix, $db, $index, $user, $cookie, $storynum, $user, $cookie, $Default_Theme, $foot1, $foot2, $foot3, $foot4, $home, $module, $name, $admin;

also find:

Code:

if ($module == 1 AND file_exists("modules/$name/admin/panel.php")) {

change to:

Code:

if ($module == 1 AND (file_exists("modules/$name/admin/panel.php") && is_admin($admin))) {

With this panel.php will only get included if it exists and the user is an admin.

mauldus · 10-14-2004, 12:53 AM

Excelent. I didn't want to have to comment out the panel function. That worked very well. Thanks a ton.