Comment 9 for bug 835103

Revision history for this message
Abel Deuring (adeuring) wrote :

Marc Tardif (cr3) wrote on 2011-08-26:

> First, I would have a look at the XML of some submissions marked as invalid. I have noticed a <udev> tag in some cases, so I would confirm this is the case.
>
> Second, assuming there is indeed a <udev> tag, I'd make sure that the schema validator supports that tag. Then, make sure that the parser actually does something with that tag.
>
> I suspect that might be a likely candidate to explain all the invalid submissions.

Marc, right, the bug is related to the <udev> node. The node itself is fine, but it does not provide enough data, as already explained. And you are right that the check against the RelaxNG schema failed.

The point of the RelaxNG check is this: consider the RelaxNG schema as a kind of a contract between checkbox and Launchpad. Launchpad has to make some assumptions about the data it can expect from the client: what sort of information does the client provide, as well as how the data is represented.

Let's make sure that a problem like this, which is trivial in its core, does not get unnoticed again for such a long time. Let's try to prvent this at its root: Let us better synchronize changes in checkbox and changes in Launchpad.