Use check-types.contains to express intent a bit better

9 years ago · 72e3ea1381
1 changed files with 1 additions and 1 deletions
--- a/sanitiser/_ids.js
+++ b/sanitiser/_ids.js
@ -48,7 +48,7 @@ function sanitize( raw, clean ){
    var id   = rawId.substring(param_index + 1);

    // check id format
-    if(param_index === -1 || !check.unemptyString( id ) || !check.unemptyString( type )) {
+    if(!check.contains(rawId, ID_DELIM) || !check.unemptyString( id ) || !check.unemptyString( type )) {
      messages.errors.push( formatError(rawId) );
    }
    // type text must be one of the types