Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.4.0

Bug #8424 incorrect handling of single column joins
Submitted: 2006-08-13 12:48 UTC
From: richard dot udo at gmail dot com Assigned: lsmith
Status: Closed Package: LiveUser_Admin
PHP Version: 5.1.4 OS: Winxp
Roadmaps: (Not assigned)    
Subscription  


 [2006-08-13 12:48 UTC] richard dot udo at gmail dot com (Richard)
Description: ------------ LiveUser_Admin / LiveUser ------------------------------ Couldnt work out a way to join multiple tables when retrieving users. Initially the user id colum of one table was authuserid liveuser_users and auth_user_ud in liveuser_perm_users tables. So i tried to a join on liveuser_users.authuserid and liveuser_perm_users.auth_user_id without much luck. From the code i investigated .... Admin/Storage/SQL.php .... around line 796 >>>>>>>>>> $filters[] = $this->prefix.$this->alias[$root_table].'.'.$this->tables[$root_table]['joins'][$table].' = '. $this->prefix.$this->alias[$table].'.'.$this->tables[$root_table]['joins'][$table]; <<<<<<<<<<<<<<<<<<<<< it doesnt look like this is possibe

Comments

 [2006-08-13 13:19 UTC] lsmith (Lukas Smith)
the following diff should probably fix the issue RCS file: /repository/pear/LiveUser_Admin/Storage/SQL.php,v retrieving revision 1.60 diff -u -r1.60 SQL.php --- SQL.php 17 Jul 2006 16:50:21 -0000 1.60 +++ SQL.php 13 Aug 2006 12:41:39 -0000 @@ -779,7 +779,7 @@ // handle single column join } else { $filters[] = $this->prefix.$this->alias[$root_table].'.'.$this->tables[$root_table]['joins'][$table].' = '. - $this->prefix.$this->alias[$table].'.'.$this->tables[$root_table]['joins'][$table]; + $this->prefix.$this->alias[$table].'.'.$this->tables[$table]['joins'][$root_table]; } unset($tables[$table]); }
 [2006-08-18 09:59 UTC] lsmith (Lukas Smith)
This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.