Merge pull request #946 from geekwright/profile_userinfo_tpl

Bad form in profile_userinfo.tpl
XOOPS · Feb 18, 2021 · 3f8cafe · 3f8cafe
2 parents 9a4e505 + 65ac0fc
commit 3f8cafe
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 28 deletions.
diff --git a/htdocs/modules/profile/templates/profile_userinfo.tpl b/htdocs/modules/profile/templates/profile_userinfo.tpl
@@ -31,15 +31,9 @@
                 <input type="button" value="<{$smarty.const._PROFILE_MA_CHANGEMAIL}>"
                        onclick="location='<{$xoops_url}>/modules/<{$xoops_dirname}>/changemail.php'"/>
             <{/if}>
-
             <{if $user_candelete == true}>
-                <form method="post" action="<{$xoops_url}>/modules/<{$xoops_dirname}>/user.php">
-                    <input type="hidden" name="op" value="delete">
-                    <input type="hidden" name="uid" value="<{$user_uid}>">
-                    <input type="button" value="<{$lang_deleteaccount}>" onclick="submit();"/>
-                </form>
+                <input type="button" value="<{$lang_deleteaccount}>" onclick="location='user.php?op=delete'">
             <{/if}>
-
             <input type="button" value="<{$lang_avatar}>" onclick="location='edituser.php?op=avatarform'"/>
             <input type="button" value="<{$lang_inbox}>" onclick="location='<{$xoops_url}>/viewpmsg.php'"/>
             <input type="button" value="<{$lang_logout}>" onclick="location='<{$xoops_url}>/modules/<{$xoops_dirname}>/user.php?op=logout'"/>

diff --git a/htdocs/themes/xbootstrap/modules/profile/profile_userinfo.tpl b/htdocs/themes/xbootstrap/modules/profile/profile_userinfo.tpl
@@ -32,15 +32,9 @@
                     <input class="btn btn-success btn-xs btn-block" type="button" value="<{$smarty.const._PROFILE_MA_CHANGEMAIL}>"
                            onclick="location='<{$xoops_url}>/modules/<{$xoops_dirname}>/changemail.php'">
                 <{/if}>
-
                 <{if $user_candelete == true}>
-                    <form method="post" action="<{$xoops_url}>/modules/<{$xoops_dirname}>/user.php">
-                        <input type="hidden" name="op" value="delete">
-                        <input type="hidden" name="uid" value="<{$user_uid}>">
-                        <input class="btn btn-success btn-xs btn-block" type="button" value="<{$lang_deleteaccount}>" onclick="submit();">
-                    </form>
+                    <input class="btn btn-success btn-xs btn-block" type="button" value="<{$lang_deleteaccount}>" onclick="location='user.php?op=delete'">
                 <{/if}>
-
                 <input class="btn btn-success btn-xs btn-block" type="button" value="<{$lang_avatar}>" onclick="location='edituser.php?op=avatarform'">
                 <input class="btn btn-success btn-xs btn-block" type="button" value="<{$lang_inbox}>" onclick="location='<{$xoops_url}>/viewpmsg.php'">
                 <input class="btn btn-success btn-xs btn-block" type="button" value="<{$lang_logout}>"

diff --git a/htdocs/themes/xswatch/modules/profile/profile_userinfo.tpl b/htdocs/themes/xswatch/modules/profile/profile_userinfo.tpl
@@ -32,15 +32,9 @@
                     <input class="btn btn-success btn-xs btn-block" type="button" value="<{$smarty.const._PROFILE_MA_CHANGEMAIL}>"
                            onclick="location='<{$xoops_url}>/modules/<{$xoops_dirname}>/changemail.php'">
                 <{/if}>
-
                 <{if $user_candelete == true}>
-                    <form method="post" action="<{$xoops_url}>/modules/<{$xoops_dirname}>/user.php">
-                        <input type="hidden" name="op" value="delete">
-                        <input type="hidden" name="uid" value="<{$user_uid}>">
-                        <input class="btn btn-success btn-xs btn-block" type="button" value="<{$lang_deleteaccount}>" onclick="submit();">
-                    </form>
+                    <input class="btn btn-success btn-xs btn-block" type="button" value="<{$lang_deleteaccount}>" onclick="location='user.php?op=delete'">
                 <{/if}>
-
                 <input class="btn btn-success btn-xs btn-block" type="button" value="<{$lang_avatar}>" onclick="location='edituser.php?op=avatarform'">
                 <input class="btn btn-success btn-xs btn-block" type="button" value="<{$lang_inbox}>" onclick="location='<{$xoops_url}>/viewpmsg.php'">
                 <input class="btn btn-success btn-xs btn-block" type="button" value="<{$lang_logout}>"

diff --git a/htdocs/themes/xswatch4/modules/profile/profile_userinfo.tpl b/htdocs/themes/xswatch4/modules/profile/profile_userinfo.tpl
@@ -32,15 +32,9 @@
                     <input class="btn btn-primary btn-xs btn-block" type="button" value="<{$smarty.const._PROFILE_MA_CHANGEMAIL}>"
                            onclick="location='<{$xoops_url}>/modules/<{$xoops_dirname}>/changemail.php'">
                 <{/if}>
-
                 <{if $user_candelete == true}>
-                    <form method="post" action="<{$xoops_url}>/modules/<{$xoops_dirname}>/user.php">
-                        <input type="hidden" name="op" value="delete">
-                        <input type="hidden" name="uid" value="<{$user_uid}>">
-                        <input class="btn btn-primary btn-xs btn-block" type="button" value="<{$lang_deleteaccount}>" onclick="submit();">
-                    </form>
+                    <input class="btn btn-primary btn-xs btn-block" type="button" value="<{$lang_deleteaccount}>" onclick="location='user.php?op=delete'">
                 <{/if}>
-
                 <input class="btn btn-primary btn-xs btn-block" type="button" value="<{$lang_avatar}>" onclick="location='edituser.php?op=avatarform'">
                 <input class="btn btn-primary btn-xs btn-block" type="button" value="<{$lang_inbox}>" onclick="location='<{$xoops_url}>/viewpmsg.php'">
                 <input class="btn btn-primary btn-xs btn-block" type="button" value="<{$lang_logout}>"