List:Eventum General Discussion« Previous MessageNext Message »
From:Georger Araujo Date:September 5 2008 9:16pm
Subject:Yet another bug report, and a request
View as plain text  
Hi,
I just downloaded Eventum's latest SVN snapshot and found a bug in the setup template. I
think it's rather serious, because it halts installation.
In line 162 of <path-to-eventum>/templates/setup.tpl.html, it reads

--8<--
            {t}In order to check if your permissions are setup correctly visit the{t}
<a class="link" href="check_permissions.php">{t}Check Permissions{/t}</a>
{t}page{/t}.
--8<--

and should be

--8<--
            {t}In order to check if your permissions are setup correctly visit the{/t}
<a class="link" href="check_permissions.php">{t}Check Permissions{/t}</a>
{t}page{/t}.
--8<--

Notice that after "visit the", there's a "{t}" where there should be a "{/t}".

Also, please consider adding a link to the login form in the anonymous reporting form. I
have implemented it at my installation, here's the diff:

--8<--
diff -u /tmp/eventum-svn/templates/post.tpl.html post.tpl.html
--- /tmp/eventum-svn/templates/post.tpl.html   2008-09-04 18:16:21.000000000 -0300
+++ post.tpl.html       2008-09-05 16:02:11.000000000 -0300
@@ -29,6 +29,11 @@
             {t 1=$smarty.server.PHP_SELF escape=no}You may <a class="link"
href="%1">submit another issue</a> if you so wish.{/t}
           </td>
         </tr>
+        <tr align="center">
+          <td colspan="2" bgcolor="{$cell_color}" class="default_white">
+            <a class="white_link" href="index.php">{t}Back to Login
Form{/t}</a>
+          </td>
+        </tr>
       </table>
     </td>
   </tr>
@@ -119,6 +124,11 @@
     <td>
       <table bgcolor="#FFFFFF" width="100%" cellspacing="1" cellpadding="2"
border="0">
         <tr>
+          <td colspan="2" bgcolor="{$cell_color}" class="default_white">
+            <a class="white_link" href="index.php">{t}Back to Login
Form{/t}</a>
+          </td>
+        </tr>
+        <tr>
           <td colspan="2" class="default">
             <b>{t}Report New Issue{/t}</b>
           </td>
--8<--

Regards,

Georger


      Novos endereços, o Yahoo! que você conhece. Crie um email novo com a
sua cara @ymail.com ou @rocketmail.com.
http://br.new.mail.yahoo.com/addresses
Thread
Yet another bug report, and a requestGeorger Araujo5 Sep