[pve-devel] [PAST pve-manager] use correct checked value for ha resource edit dialog

Thomas Lamprecht t.lamprecht at proxmox.com
Wed Sep 9 18:12:43 CEST 2015


Didn't replied to the list, sorry.

-------- Weitergeleitete Nachricht --------
Betreff: 	Re: [pve-devel] [PAST pve-manager] use correct checked value 
for ha resource edit dialog
Datum: 	Wed, 9 Sep 2015 18:10:50 +0200
Von: 	Thomas Lamprecht <t.lamprecht at proxmox.com>
An: 	Dietmar Maurer <dietmar at proxmox.com>



Reproduction:
Disable an service and then try to edit it. You cannot enable it
directly as the checkbox is already checked.
So you need to save it as disabled (although it is already disabled)
reopen the edit box and only then you can enable it.

Am 09.09.2015 um 17:31 schrieb Dietmar Maurer:
> that looks strange to me - is that really required?
> The current GUI works for me, so how can I reproduce the bug?
>
>> On September 9, 2015 at 3:21 PM Thomas Lamprecht <t.lamprecht at proxmox.com>
>> wrote:
>>
>>
>> Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
>> ---
>>   www/manager/ha/ResourceEdit.js | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/www/manager/ha/ResourceEdit.js b/www/manager/ha/ResourceEdit.js
>> index 962cdda..bc48c70 100644
>> --- a/www/manager/ha/ResourceEdit.js
>> +++ b/www/manager/ha/ResourceEdit.js
>> @@ -54,7 +54,7 @@ Ext.define('PVE.ha.VMResourceInputPanel', {
>>   	    {
>>   		xtype: 'pvecheckbox',
>>   		name: 'enable',
>> -		checked: true,
>> +		checked: me.enable,
>>   		uncheckedValue: 0,
>>   		fieldLabel: gettext('enable')
>>   	    }
>> --
>> 2.1.4
>>
>> _______________________________________________
>> pve-devel mailing list
>> pve-devel at pve.proxmox.com
>> http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
>>



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.proxmox.com/pipermail/pve-devel/attachments/20150909/5dcdd815/attachment.htm>


More information about the pve-devel mailing list