#165 XOOPS 2.3.0 System Templates - xHTML corrected

[Steve Kenow]

I have modified the 2.3.0 system module templates, correcting some of the xHTML for validation purposes -

system_comments_flat.html
system_comments_nest.html
system_comments_thread.html
system_imagemanager2.html
system_imagemanager.html
system_redirect.html
system_siteclosed.html
system_userinfo.html

blocks/
system_block_comments.html
system_block_newusers.html
system_block_siteinfo.html
system_block_topusers.html

[rowd]

Logged In: YES
user_id=1095348
Originator: NO

Nice work! I attached a diff file to make it easier to review.

The only tiny detail that imo should be changed is the white space in system_imagemanager.html line #51 and system_imagemanager2.html line #32.

Other than that for me it's fine. If Skalpa (the release manager) agrees then it will be in the next release.

Thanks for helping to improve XOOPS :-)

File Added: patch-1712443-diff.zip

[aphex]

Logged In: YES
user_id=1817010
Originator: NO

Files where revised and one line in one file fixed "system_userinfo.html"

zip file rebuild on http://templates.pierdtimp.ro/xoops230templatechanges.zip

can one add this file to the tracker please

Thank you

[Steve Kenow]

updated to include a-ph3x's change + a few more

[Steve Kenow]

Logged In: YES
user_id=773222
Originator: YES

File Added: xoops230system.zip

[Steve Kenow]

Logged In: YES
user_id=773222
Originator: YES

File Added: templates.diff

[Steve Kenow]

New diff file between patch and 2.0.17

[D.J.]

Logged In: YES
user_id=1001493
Originator: NO

Next week

[D.J.]

Logged In: YES
user_id=1001493
Originator: NO

OK, I'll take care of this one personally.

[aphex]

Logged In: YES
user_id=1817010
Originator: NO

now thats what i call good news :D

[Kris_fr]

Logged In: YES
user_id=1730840
Originator: NO

good exemple --> system_block_siteinfo.html

---

<{if $block.showgroups}>

<{foreach item=group from=$block.groups}> <{if $group.users}>
• <{$group.name}>
  <{foreach from=$group.users item=user}>
  • <{$user.name}>
    <{$user.msglink}>
  <{/foreach}>
<{/if}> <{/foreach}>

<{/if}>

<{$block.recommendlink}>

---

or --> system_notification_select.html

---

<{$xoTheme->addStylesheet('modules/system/style.css')}>
<script type="text/javascript">
function hideAll()
{
document.getElementById('notifform').style.display = 'none';
}
function showForm(obj)
{
obj.style.display = 'block';
}
</script>

<style type="text/css"> #notifform { display: none;} </style>

<{if $xoops_notification.show}>

 
  
  <{$lang_notificationoptions}>

<form id="notifform" name="notification_select" action="&lt;{$xoops_notification.target_page}&gt;" method="post">

<{$lang_activenotifications}>

<input type="hidden" name="not_redirect" value="&lt;{$xoops_notification.redirect_script}&gt;"/> <{foreach name=outer item=category from=$xoops_notification.categories}> <{foreach name=inner item=event from=$category.events}> <{if $smarty.foreach.inner.first}> <{/if}> <{/foreach}> <{/foreach}>

<{$lang_notificationoptions}>

<{$lang_category}>	<input name="allbox" id="allbox" onclick="xoopsCheckAll('notification_select','allbox');" type="checkbox" class="checkbox" value="&lt;{$lang_checkall}&gt;"/>	<{$lang_events}>
<input type="submit" class="button" name="not_submit" value="&lt;{$lang_updatenow}&gt;"/>
<{$category.title}>	<{counter assign=index}> <input type="hidden" name="not_list[&lt;{$index}&gt;][params]" value="&lt;{$category.name}&gt;,&lt;{$category.itemid}&gt;,&lt;{$event.name}&gt;"/> <input type="checkbox" class="checkbox" id="not_list[]" name="not_list[&lt;{$index}&gt;][status]" value="1" <{if="" $event.subscribed}="">checked="checked"<{/if}>	<{$event.caption}>

<{$lang_notificationmethodis}>: <{$user_method}>  [<{$lang_change}>]

</form>

<{/if}>

[Kris_fr]

Logged In: YES
user_id=1730840
Originator: NO

And for system_userinfo.html (exemple version 2.0.1x) :

---

<{if $user_ownpage == true}> <form name="usernav" action="user.php" method="post">

<input type="button" value="&lt;{$lang_editprofile}&gt;" onclick="location='edituser.php'" title="&lt;{$lang_editprofile}&gt;"/> <input type="button" value="&lt;{$lang_avatar}&gt;" onclick="location='edituser.php?op=avatarform'" title="&lt;{$lang_avatar}&gt;"/> <input type="button" value="&lt;{$lang_inbox}&gt;" onclick="location='viewpmsg.php'" title="&lt;{$lang_inbox}&gt;"/> <{if $user_candelete == true}> <input type="button" value="&lt;{$lang_deleteaccount}&gt;" onclick="location='user.php?op=delete'" title="&lt;{$lang_deleteaccount}&gt;"/> <{/if}> <input type="button" value="&lt;{$lang_logout}&gt;" onclick="location='user.php?op=logout'" title="&lt;{$lang_logout}&gt;"/>

</form>

<{elseif $xoops_isadmin != false}>

<input type="button" value="&lt;{$lang_editprofile}&gt;" onclick="location='&lt;{$xoops_url}&gt;/modules/system/admin.php?fct=users&amp;uid=&lt;{$user_uid}&gt;&amp;op=modifyUser'" title="&lt;{$lang_editprofile}&gt;"/> <input type="button" value="&lt;{$lang_deleteaccount}&gt;" onclick="location='&lt;{$xoops_url}&gt;/modules/system/admin.php?fct=users&amp;op=delUser&amp;uid=&lt;{$user_uid}&gt;'" title="&lt;{$lang_deleteaccount}&gt;"/>

<{/if}>

<{$lang_allaboutuser}> <{$lang_avatar}> <{$lang_realname}> <{$user_realname}> <{$lang_website}> <{$user_websiteurl}> <{$lang_email}> <{$user_email}> <{$lang_privmsg}> <{$user_pmlink}> <{$lang_icq}> <{$user_icq}> <{$lang_aim}> <{$user_aim}> <{$lang_yim}> <{$user_yim}> <{$lang_msnm}> <{$user_msnm}> <{$lang_location}> <{$user_location}> <{$lang_occupation}> <{$user_occupation}> <{$lang_interest}> <{$user_interest}> <{$lang_extrainfo}> <{$user_extrainfo}>

<{$lang_statistics}> <{$lang_membersince}> <{$user_joindate}> <{$lang_rank}> <{$user_rankimage}> <{$user_ranktitle}> <{$lang_posts}> <{$user_posts}> <{$lang_lastlogin}> <{$user_lastlogin}> <{$lang_signature}> <{$user_signature}>

<{foreach item=module from=$modules}>

<{$module.name}>

<{foreach item=result from=$module.results}>   <{$result.title}>
(<{$result.time}>)
<{/foreach}> <{$module.showall_link}>

<{/foreach}>