δΊ 2014/2/21 17:15, Mark Wu ει:
The current regex will refuse the strings which end with '/'. Obviously, even if the dir path ends with '/', it's still valid. So this patch changes the regex to accept this case.
Signed-off-by: Mark Wu <wudxw@linux.vnet.ibm.com> --- ui/js/src/kimchi.storagepool_add_main.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ui/js/src/kimchi.storagepool_add_main.js b/ui/js/src/kimchi.storagepool_add_main.js index ac97e1a..8c0025f 100644 --- a/ui/js/src/kimchi.storagepool_add_main.js +++ b/ui/js/src/kimchi.storagepool_add_main.js @@ -167,7 +167,7 @@ kimchi.validateDirForm = function () { kimchi.message.error.code('KCHPOOL6002E'); return false; } - if (!/((\/([0-9a-zA-Z-_\.]+)))$/.test(path)) { + if (!/(\/([0-9a-zA-Z-_\.]+))+\/?$/.test(path)) { I think "/foo1/dev///" is also a legal path.
kimchi.message.error.code('KCHAPI6003E'); return false; }