Tickets :: [#7916] \U does not behave as described in the custom sql driver

6.0.0-beta1

7/8/25

Summary	\U does not behave as described in the custom sql driver
Queue	Vacation
Queue Version	3.1-RC1
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	jason.stumpf (at) gmail (dot) com
Created	01/29/2009 (6004 days ago)
Due
Updated	08/18/2009 (5803 days ago)
Assigned	01/29/2009 (6004 days ago)
Resolved	08/18/2009 (5803 days ago)
Milestone	3.1.1
Patch	No

08/18/2009 12:55:42 PM	Jan Schneider	Comment #9 Assigned to Jan Schneider State ⇒ Resolved	Reply to this comment
I did the same for forwards too.

08/18/2009 12:54:51 PM	CVS Commit	Comment #8	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/forwards/config/conf.xml?rt=horde&r1=1.7.2.3&r2=1.7.2.4&ty=u http://cvs.horde.org/diff.php/forwards/docs/CHANGES?rt=horde&r1=1.34.2.14&r2=1.34.2.15&ty=u http://cvs.horde.org/diff.php/forwards/lib/Driver/customsql.php?rt=horde&r1=1.13.2.6&r2=1.13.2.7&ty=u http://cvs.horde.org/diff.php/vacation/config/conf.xml?rt=horde&r1=1.15.2.12&r2=1.15.2.13&ty=u http://cvs.horde.org/diff.php/vacation/docs/CHANGES?rt=horde&r1=1.57.2.21&r2=1.57.2.22&ty=u http://cvs.horde.org/diff.php/vacation/lib/Driver/customsql.php?rt=horde&r1=1.2.2.5&r2=1.2.2.6&ty=u

08/18/2009 12:52:14 PM	CVS Commit	Comment #7	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/forwards/config/conf.xml?rt=horde&r1=1.15&r2=1.16&ty=u http://cvs.horde.org/diff.php/forwards/docs/CHANGES?rt=horde&r1=1.47&r2=1.48&ty=u http://cvs.horde.org/diff.php/forwards/lib/Driver/customsql.php?rt=horde&r1=1.20&r2=1.21&ty=u http://cvs.horde.org/diff.php/vacation/config/conf.xml?rt=horde&r1=1.34&r2=1.35&ty=u http://cvs.horde.org/diff.php/vacation/docs/CHANGES?rt=horde&r1=1.79&r2=1.80&ty=u http://cvs.horde.org/diff.php/vacation/lib/Driver/customsql.php?rt=horde&r1=1.6&r2=1.7&ty=u

03/19/2009 03:05:41 PM	Jan Schneider	Milestone ⇒ 3.1.1

01/29/2009 10:17:44 PM	jason (dot) stumpf (at) gmail (dot) com	Comment #6	Reply to this comment
Sorry, I didn't refresh before commenting... didn't see your comments.

01/29/2009 10:15:40 PM	jason (dot) stumpf (at) gmail (dot) com	Comment #5	Reply to this comment
This fixes it at least for me: in Driver.php where it says: function Vacation_Driver($user, $realm, $params = array()) { $this->_params = $params; // Check if hordeauth is set to 'full' $hordeauth = $this->getParam('hordeauth'); if ($hordeauth !== 'full') { @list($user,) = explode('@', $user, 2); } $this->_user = $user; if (!isset($params[$realm])) { $realm = 'default'; } $this->_realm = $realm; } change it to: function Vacation_Driver($user, $realm, $params = array()) { $this->_params = $params; if (!isset($params[$realm])) { $realm = 'default'; } $this->_realm = $realm; // Check if hordeauth is set to 'full' $hordeauth = $this->getParam('hordeauth'); if ($hordeauth !== 'full') { @list($user,) = explode('@', $user, 2); } $this->_user = $user; } I don't know your codebase well, but it seems to me that getParam depends on $this->_ream being set, but it wasn't.

01/29/2009 10:10:31 PM	Jan Schneider	Taken from Jan Schneider Milestone ⇒ 3.1

01/29/2009 10:10:14 PM	Jan Schneider	Comment #4 State ⇒ Accepted Type ⇒ Enhancement Priority ⇒ 1. Low	Reply to this comment
I think we need different placeholders for full user name and local user name, i.e. username without domain or with realm suffix.

01/29/2009 10:03:08 PM	Jan Schneider	Comment #3 Assigned to Jan Schneider State ⇒ Assigned	Reply to this comment
Same in the sql driver. Also, it doesn't seem to care what I set the hordeauth parameter to. The latter has been fixed in CVS already.

01/29/2009 09:47:04 PM	jason (dot) stumpf (at) gmail (dot) com	Comment #2	Reply to this comment
Same in the sql driver. Also, it doesn't seem to care what I set the hordeauth parameter to.

01/29/2009 09:36:05 PM	jason (dot) stumpf (at) gmail (dot) com	Comment #1 Priority ⇒ 1. Low State ⇒ Unconfirmed Patch ⇒ No Milestone ⇒ Queue ⇒ Vacation Summary ⇒ \U does not behave as described in the custom sql driver Type ⇒ Bug	Reply to this comment
I am using vacation-FRAMEWORK_3-2009-01-29.tar.gz The config for custom sql says "The current FULL username will be provided as \U", but actually only the username up to the @ is provided. (I know this could be fixed by just changing the description, but at least to me, changing the behavior to give the full username would be much more helpful.)