HomePhabricator

Relax Lexeme deserialization to not fail with $nextFormId = 1
72ba23b59cc2Unpublished

Unpublished Commit · Learn More

Not On Permanent Ref: This commit is not an ancestor of any permanent ref.
This commit no longer exists in the repository. It may have been part of a branch which was deleted.This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref.

Description

Relax Lexeme deserialization to not fail with $nextFormId = 1

With the default set to 1 this was guaranteed to fail. See
http://wikidata-lexeme.wmflabs.org/index.php/Lexeme:L25
for an example why this is relevant. We might make this more strict
later, but for now we figured it helps to relax this.

Also, if the nextFormId field is not given, but forms are given, why
should the code always fail in such a situation?

Bug: T166691
Change-Id: If2e4afe80a977ef5cdbbe4e68f329ef7191a3e62

Details

Provenance
thiemowmdeAuthored on Oct 12 2017, 7:49 AM
ChangeId
If2e4afe80a977ef5cdbbe4e68f329ef7191a3e62

Commit No Longer Exists

This commit no longer exists in the repository.