From: Pascal Corpet <pascal.corpet@m4x.org>
Date: Tue, 8 Jun 2010 20:59:42 +0000 (+0200)
Subject: Fixes accessing to groups having shortnames that look like ids. Closes #1084
X-Git-Tag: xorg/1.0.0~144
X-Git-Url: http://git.polytechnique.org/?a=commitdiff_plain;h=f6e35ff3fefafe908e1675690340ab93265192a0;p=platal.git

Fixes accessing to groups having shortnames that look like ids. Closes #1084
---

diff --git a/classes/group.php b/classes/group.php
index f58ff6b..e3f3b62 100644
--- a/classes/group.php
+++ b/classes/group.php
@@ -93,15 +93,15 @@ class Group
         return null;
     }
 
-    static public function get($id)
+    static public function get($id, $can_be_shortname = true)
     {
         if (!$id) {
             return null;
         }
-        if (is_int($id) || ctype_digit($id)) {
-            $where = XDB::format('id = {?}', $id);
+        if (!$can_be_shortname) {
+            $where = XDB::format('a.id = {?}', $id);
         } else {
-            $where = XDB::format('diminutif = {?}', $id);
+            $where = XDB::format('a.diminutif = {?}', $id);
         }
         $res = XDB::query('SELECT  a.*, d.nom AS domnom,
                                    FIND_IN_SET(\'wiki_desc\', a.flags) AS wiki_desc,
@@ -110,6 +110,9 @@ class Group
                         LEFT JOIN  group_dom  AS d ON d.id = a.dom
                             WHERE  ' . $where);
         if ($res->numRows() != 1) {
+            if ($can_be_shortname && (is_int($id) || ctype_digit($id))) {
+                return Group::get($id, false);
+            }
             return null;
         }
         return new Group($res->fetchOneAssoc());