ext5migrate: do not set a custome idProperty for the KeyValue model

ExtJS refuses with version to have an id set to an empty string, hence we can't
use our "key" property as a custom idProperty (an empty string for key is ok for us, and is used to set back a PVE property to its default value)

We always access the KeyValues entities via their Key property, so this change should be safe.

diff --git a/www/manager5/data/PVEProxy.js b/www/manager5/data/PVEProxy.js
index 20c1acbb521ecc557fafda91ff1060b92fc0fdf4..01c20cb08b2d6ad8c3c6ce11592c2be73dc99e83 100644 (file)
--- a/www/manager5/data/PVEProxy.js
+++ b/www/manager5/data/PVEProxy.js
@@ -61,8 +61,7 @@ Ext.define('PVE.RestProxy', {
 
     Ext.define('KeyValue', {
        extend: "Ext.data.Model",
-       fields: [ 'key', 'value' ],
-       idProperty: 'key'
+       fields: [ 'key', 'value' ]
     });
 
     Ext.define('KeyValuePendingDelete', {