Revision ID wrong

[jimbowhales]

In cases like

<revision>
      <id>73841108</id>
      <parentid>73290874</parentid>
      <timestamp>2019-08-14T00:23:07Z</timestamp>
      <contributor>
        <username>Ttle-recll</username>
        <id>1525028</id>
      </contributor>

WikiExtractor takes the contributor ID as revision ID. A likely fix would be to change

        elif tag == 'id' and not id:
            id = m.group(3)
        elif tag == 'id' and id:
            revid = m.group(3)

to

        elif tag == 'id' and not id:
            id = m.group(3)
        elif tag == 'id' and not revid:
            revid = m.group(3)

but I haven't tested if this fails in some other case.

[HjalmarrSv]

Thanx!

Looks like an improvement. It should work nicely as long as the tag order is not changed, or new tags inserted before this. The downside of a partially implemented xml reader is that the order matters.

[attardi]

Accepted.