Merge pull request #124 from mormegil-cz/bugfix/deleting-aliases

CXuesong · web-flow · commit e1e45c1ebe8a · 2025-10-18T10:58:22.000+08:00
When removing aliases, the old value needs to be sent to the API
diff --git a/WikiClientLibrary.Wikibase/Entity.Editing.cs b/WikiClientLibrary.Wikibase/Entity.Editing.cs
@@ -60,7 +60,7 @@ private Contracts.Entity SerializeEditEntries(IEnumerable<EntityEditEntry> edits
                         Debug.Assert(entry.Value != null);
                         var value = (WbMonolingualText)entry.Value;
                         var item = entry.State == EntityEditEntryState.Removed
-                            ? new Contracts.MonolingualText { Language = value.Language, Remove = true }
+                            ? new Contracts.MonolingualText { Language = value.Language, Value = value.Text, Remove = true }
                             : new Contracts.MonolingualText { Language = value.Language, Value = value.Text };
                         if (!contract.Aliases.TryGetValue(item.Language, out var items))
                         {

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ private Contracts.Entity SerializeEditEntries(IEnumerable<EntityEditEntry> edits`
`60`	`60`	`Debug.Assert(entry.Value != null);`
`61`	`61`	`var value = (WbMonolingualText)entry.Value;`
`62`	`62`	`var item = entry.State == EntityEditEntryState.Removed`
`63`		`- ? new Contracts.MonolingualText { Language = value.Language, Remove = true }`
	`63`	`+ ? new Contracts.MonolingualText { Language = value.Language, Value = value.Text, Remove = true }`
`64`	`64`	`: new Contracts.MonolingualText { Language = value.Language, Value = value.Text };`
`65`	`65`	`if (!contract.Aliases.TryGetValue(item.Language, out var items))`
`66`	`66`	`{`