[
https://issues.apache.org/jira/browse/OFBIZ-3071?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12777446#action_12777446
]
Abdullah Shaikh commented on OFBIZ-3071:
----------------------------------------
Good to hear that it makes sense ..
Even if we don't constrain that it should work just fine, we wont have any
trouble.
But what I feel is if we have found the issue then lets just fix it, only
because it wont cause any trouble or it's only redundant that will do, will
accumulate more and more of this kind of issues and maybe later we will start
finding this kind of issues and fixing it later.
As Ofbiz is an ERP it maybe grow more & more bigger, it's better if we fix the
issues irrespective of how big or small the issue is.
I dont remember but I guess the patch was handling more than this ..
If you feel that we can commit the thruDate fix, then I can work on the patch
to just fix this issue and remove the other stuff, let me know.
> assigning security group to a party without validations
> -------------------------------------------------------
>
> Key: OFBIZ-3071
> URL: https://issues.apache.org/jira/browse/OFBIZ-3071
> Project: OFBiz
> Issue Type: Improvement
> Reporter: Abdullah Shaikh
> Priority: Minor
> Attachments: OFBIZ-3071_assigning security group to a party without
> validations.patch
>
>
> When adding user login to Security Group, the fromDate & thruDate input
> fields are not mandatory, if not specified the fromDate is set to current
> timestamop and thruDate is entered in database as null, which means the user
> login is assigned that Security Group forever, but when if the same Security
> Group is added, it again adds it, but instead it should check that if the
> thruDate is null in the already entered record then shouldn't add new record.
> If incase the thruDate is specified during the creation of 1st record then
> while adding the 2nd record for the same Security Group, it should check if
> the thruDate of previous record is before the fromDate of the new record,
> this check also doesn't happen.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.