You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Michael Brohl (Created) (JIRA)" <ji...@apache.org> on 2012/02/14 18:05:59 UTC
[jira] [Created] (OFBIZ-4693) return value of
currentDelegatorTenantId.trim() ignored in
org.ofbiz.webapp.control.LoginWorker.login(HttpServletRequest,
HttpServletResponse)
return value of currentDelegatorTenantId.trim() ignored in org.ofbiz.webapp.control.LoginWorker.login(HttpServletRequest, HttpServletResponse)
----------------------------------------------------------------------------------------------------------------------------------------------
Key: OFBIZ-4693
URL: https://issues.apache.org/jira/browse/OFBIZ-4693
Project: OFBiz
Issue Type: Bug
Components: framework
Affects Versions: SVN trunk
Reporter: Michael Brohl
The return value of currentDelegatorTenantId.trim() is ignored in org.ofbiz.webapp.control.LoginWorker.login(HttpServletRequest, HttpServletResponse) so trim() has no effect.
I will provide a patch.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Closed] (OFBIZ-4693) return value of
currentDelegatorTenantId.trim() ignored in
org.ofbiz.webapp.control.LoginWorker.login(HttpServletRequest,
HttpServletResponse)
Posted by "Jacques Le Roux (Closed) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-4693?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jacques Le Roux closed OFBIZ-4693.
----------------------------------
Resolution: Fixed
Fix Version/s: SVN trunk
Release Branch 11.04
Release Branch 10.04
Assignee: Jacques Le Roux
Thanks Michael,
Your patch is in
trunk r1244139
R11.04 r1244140
R10.04 r1244148
It was easily applied (fortunately very simple) but not done following the rules https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+Contributors+Best+Practices
Notably a lot of false changes and not done from root folder.
For the false changes no pb it was only "trailing" blanks removed (at least personnally I'm not agains, thanks to Subclipse which hides them). But then you should better tell us. I know that sometimes it's a pain to have to switch this setting on/off... Personally I tried to avoid those changes when it really complicates the patch. This one was not.
> return value of currentDelegatorTenantId.trim() ignored in org.ofbiz.webapp.control.LoginWorker.login(HttpServletRequest, HttpServletResponse)
> ----------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: OFBIZ-4693
> URL: https://issues.apache.org/jira/browse/OFBIZ-4693
> Project: OFBiz
> Issue Type: Bug
> Components: framework
> Affects Versions: SVN trunk
> Reporter: Michael Brohl
> Assignee: Jacques Le Roux
> Labels: patch
> Fix For: Release Branch 10.04, Release Branch 11.04, SVN trunk
>
> Attachments: LoginWorker.java.patch
>
>
> The return value of currentDelegatorTenantId.trim() is ignored in org.ofbiz.webapp.control.LoginWorker.login(HttpServletRequest, HttpServletResponse) so trim() has no effect.
> I will provide a patch.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (OFBIZ-4693) return value of
currentDelegatorTenantId.trim() ignored in
org.ofbiz.webapp.control.LoginWorker.login(HttpServletRequest,
HttpServletResponse)
Posted by "Michael Brohl (Updated) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-4693?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Michael Brohl updated OFBIZ-4693:
---------------------------------
Attachment: LoginWorker.java.patch
Also removed trailing whitespaces, I hope this is ok.
> return value of currentDelegatorTenantId.trim() ignored in org.ofbiz.webapp.control.LoginWorker.login(HttpServletRequest, HttpServletResponse)
> ----------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: OFBIZ-4693
> URL: https://issues.apache.org/jira/browse/OFBIZ-4693
> Project: OFBiz
> Issue Type: Bug
> Components: framework
> Affects Versions: SVN trunk
> Reporter: Michael Brohl
> Labels: patch
> Attachments: LoginWorker.java.patch
>
>
> The return value of currentDelegatorTenantId.trim() is ignored in org.ofbiz.webapp.control.LoginWorker.login(HttpServletRequest, HttpServletResponse) so trim() has no effect.
> I will provide a patch.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (OFBIZ-4693) return value of
currentDelegatorTenantId.trim() ignored in
org.ofbiz.webapp.control.LoginWorker.login(HttpServletRequest,
HttpServletResponse)
Posted by "Jacques Le Roux (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-4693?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13207923#comment-13207923 ]
Jacques Le Roux commented on OFBIZ-4693:
----------------------------------------
Sorry I did not read your comment <<Also removed trailing whitespaces, I hope this is ok.>>. Forget my <<But then you should better tell us.>> ;)
So to answer that: yes it was globally OK, but is not a recommended way of doing (because of many false changes to track). As I said, when patching with Subclipse it's not a pb for me, as it hides all of them (maybe I have a setting to ignore white spaces changes there).
> return value of currentDelegatorTenantId.trim() ignored in org.ofbiz.webapp.control.LoginWorker.login(HttpServletRequest, HttpServletResponse)
> ----------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: OFBIZ-4693
> URL: https://issues.apache.org/jira/browse/OFBIZ-4693
> Project: OFBiz
> Issue Type: Bug
> Components: framework
> Affects Versions: SVN trunk
> Reporter: Michael Brohl
> Assignee: Jacques Le Roux
> Labels: patch
> Fix For: Release Branch 10.04, Release Branch 11.04, SVN trunk
>
> Attachments: LoginWorker.java.patch
>
>
> The return value of currentDelegatorTenantId.trim() is ignored in org.ofbiz.webapp.control.LoginWorker.login(HttpServletRequest, HttpServletResponse) so trim() has no effect.
> I will provide a patch.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira