1a3019fbfddcca846d3db58251b9cc423417416b galt Thu Feb 23 17:32:59 2017 -0800 Fixes htmlCheckLinks (really htmlPage.c) which is demainding a name if the onchange= does not exist. I have added that also having an id= is enough to consider it not an error. refs #18918. diff --git src/lib/htmlPage.c src/lib/htmlPage.c index eb92980..8cce92f 100644 --- src/lib/htmlPage.c +++ src/lib/htmlPage.c @@ -711,30 +711,31 @@ struct htmlFormVar *varList = NULL, *var; struct hash *hash = newHash(0); for (tag = form->startTag->next; tag != form->endTag; tag = tag->next) { if (sameWord(tag->name, "INPUT")) { char *type = htmlTagAttributeVal(page, tag, "TYPE", NULL); char *varName = htmlTagAttributeVal(page, tag, "NAME", NULL); char *value = htmlTagAttributeVal(page, tag, "VALUE", NULL); if (type == NULL) type = "TEXT"; if (varName == NULL) { if (!htmlTagAttributeVal(page, tag, "ONCHANGE", NULL) + && !htmlTagAttributeVal(page, tag, "ID", NULL) && !sameWord(type, "SUBMIT") && !sameWord(type, "CLEAR") && !sameWord(type, "BUTTON") && !sameWord(type, "RESET") && !sameWord(type, "IMAGE")) tagWarn(page, tag, "Missing NAME attribute"); varName = "n/a"; } var = findOrMakeVar(page, varName, hash, tag, &varList); if (var->type != NULL && !sameWord(var->type, type)) { if (!areMixableInputTypes(var->type, type)) tagWarn(page, tag, "Mixing input types %s and %s", var->type, type); } var->type = type; if (sameWord(type, "TEXT") || sameWord(type, "PASSWORD") || sameWord(type, "FILE") || sameWord(type, "HIDDEN")