Proxmox/proxmox-widget-toolkit
5
0
Fork 0
Code Pull Requests Releases Activity

override Ext.Components 'validIdRe' to include '@'

Browse Source 
we use itemIds with '@' already in the ceph dashboard, and in
non-debug modeit works (as expected) but throws an error
while using ext-all-debug.js so override it to include the '@'

the regex was in place because in html4 the 'id' tag could only consist
letters ofthe mentioned regex, this was lifted in html5 though[0]

0: https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/id

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
Dominik Csapak 2019-07-11 14:10:30 +02:00 committed by Thomas Lamprecht
parent 227fb07f0c
commit 312310e0f2
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Toolkit.js
View File

@ -439,6 +439,12 @@ Ext.define(null, {
}); });
// add '@' to the valid id
Ext.define('Proxmox.validIdReOverride', {
override: 'Ext.Component',
validIdRe: /^[a-z_][a-z0-9\-_\@]*$/i,
});
// force alert boxes to be rendered with an Error Icon // force alert boxes to be rendered with an Error Icon
// since Ext.Msg is an object and not a prototype, we need to override it // since Ext.Msg is an object and not a prototype, we need to override it
// after the framework has been initiated // after the framework has been initiated