Welcome to Geeklog, Anonymous Sunday, October 13 2024 @ 12:07 pm EDT
Geeklog Forums
moderation.php problem???
bong
$retval .= LB . $table . LB
. '<input type="hidden" name="type" value="' . $type . '">' . LB
. '<input type="hidden" name="mode" value="moderation">' . LB
. '<input type="hidden" name="count" value="' . $nrows . '">'
. '<p align="center"><input type="submit" value="'
. $LANG_ADMIN['submit'] . '"></p></form>' . LB;
} else {
$retval .= $table;
}
Dirk
Not sure how you've got the W3C validator to see moderation.php in the first place, though. Since it can't log in, it would only see an "access denied" message ...
bye, Dirk
mevans
I just ran into an issue (while working on a theme) where ADMIN_simpleList() seems to have a problem. It is used to generate the site stats screen. Problem is, each stat block has a <form> tag, but none of them are ever closed. So on the Site Statistics screen you have a whole bunch of open <form> tags with no closing ones.
I'm willing to bet this is what bong was seeing too. His solution was probably to remove the opening form tag from admin/lists/list.thtml, which fixes the stats screen but then breaks all the admin / moderation screens.
Maybe if the form is going to be opened in the templates, it needs to be closed there as well instead of in the code. Or, possibly two list building functions, one for lists without forms and one with forms?
But, there is definitely an issue in stats.php as it stands right now.
Thanks!
Mark
Dirk
bye, Dirk
bong
thank you for good explanation. certainly, i removed it from admin/lists/list.thtml, as you said. i wish this issue will be solved to make geeklog better.
thanks, bong
tokyoahead
Not sure how you've got the W3C validator to see moderation.php in the first place, though. Since it can't log in, it would only see an "access denied" message ...
You can post the resulting HTML into a form of the validator. Opera has a function built-in to do that for example.
tokyoahead
Dirk,
I just ran into an issue (while working on a theme) where ADMIN_simpleList() seems to have a problem. It is used to generate the site stats screen. Problem is, each stat block has a <form> tag, but none of them are ever closed. So on the Site Statistics screen you have a whole bunch of open <form> tags with no closing ones.
Maybe if the form is going to be opened in the templates, it needs to be closed there as well instead of in the code. Or, possibly two list building functions, one for lists without forms and one with forms?
This was fixed in CVS already some time ago.
This is the current CVS version of the list.thtml:
{top_menu}
<form action="{form_url}" method="post">
{formfields_top}
<table class="admin-list-table">
<tr style="background:#FFFFFF;">
{header_row}
</tr>
{item_row}
</table>
<div style="padding:5px 0px 0px 9px;{show_deleteimage}">
<input name="delbutton" type="image" src="{layout_url}/images/deleteitem.png" title="{lang_deleteall}" onclick="return confirm('{lang_delconfirm}');">
</div>
<div>
{formfields_bottom}
</div>
</form>
<p class="aligncenter" style="{show_message}"><b>{message}</b></p>
{google_paging}
bong
end tag for element "FORM" which is not open.
…pe="submit" value="Submit"></p></form>
mevans
I'm not sure I would call that fixed. The issue is in stats.php, it uses ADMIN_simpleList(). But, it does not set {form_url} and also it does not set the {lang_delconfirm} either.
I haven't look in CVS to see if stats.php now uses something different than ADMIN_simpleList(). But if it is still using it, you have <form> with no action URL, incomplete 'delete' input. But, at least the form will be closed
Thanks!
Mark
tokyoahead
tokyoahead,
I'm not sure I would call that fixed. You have <form> with no action URL, incomplete 'delete' input. But, at least the form will be closed
Thanks!
Mark
well the HTML validates :-)
I know this is not a perfect solution. The issues with this form are so intricate I do not know where to start to explain. The form is used in many places in many different applications. I can make the html optional in case the form variables are not filled.
I wonder if it matters if the HTML validates.
Oliver
tokyoahead
i changed the list.thtml to current version, and checked the html with validater again, but the error is found. do i need any files besides list.thtml?
Sorry I should have explained. I simply wanted to state that the problem will be fixed in the next version since it is in CVS. You cannot fix it easily on the current version, you would have to change at least 5 different files to do that, you rather wait for the validation until the next version. sorry for that. I hope however that the validation of administrator screens are something you can live with until then :-)
- Normal Topic
- Sticky Topic
- Locked Topic
- New Post
- Sticky Topic W/ New Post
- Locked Topic W/ New Post
- View Anonymous Posts
- Able to post
- Filtered HTML Allowed
- Censored Content