[ 
https://issues.apache.org/jira/browse/OFBIZ-10367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16491638#comment-16491638
 ] 

Rishi Solanki commented on OFBIZ-10367:
---------------------------------------

[~pawan.verma]: Thanks for reporting and providing the nice enhancement. 
Reviewed your patch and it looks good to me. 
One input from my side, I see only getDisabled() method in place and no setter 
method for that. We can either add an setter method so that code will have 
flexibility to set it true/false on demand or we can make the disabled variable 
as final with no default value so that it will initialize only once from 
constructor only.

Please let me know if you have any questions?

> Add Support for Disable attribute in CheckBox Form Widget
> ---------------------------------------------------------
>
>                 Key: OFBIZ-10367
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10367
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: Trunk
>            Reporter: Pawan Verma
>            Assignee: Rishi Solanki
>            Priority: Minor
>         Attachments: OFBIZ-10367.patch
>
>
> While working on the Inventory Transfer improvements, I have found that there 
> should be support in CheckBox form-widget to disable a checkbox. There might 
> be several use cases for this, I am listing one below:
> Use Case: A screen list all the requirement out of which only Approved 
> Requirement is allowed to perform some action and other should not be allowed 
> to select. 
> To support this type of case there should be a support to disable a 
> form-widget checkbox.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to