SuLab / WikidataIntegrator

A Wikidata Python module integrating the MediaWiki API and the Wikidata SPARQL endpoint
MIT License
244 stars 46 forks source link

item_name doesn't do anything #73

Closed stuppie closed 5 years ago

stuppie commented 6 years ago

item_name isn't really used (its only checked to be not empty), but isn't used to retrieve an item or set its label. Probably confusing to new users and should be removed.

Samwalton9 commented 5 years ago

I was stumped for a while today about both this and domain. Checked the code and it seems to do nothing, so agree with removal if it's not used.

andrawaag commented 5 years ago

It actually does more than only checking to be not empty. If it is empty, together with the domain argument a new Wikidata item is issued:

https://github.com/SuLab/WikidataIntegrator/blob/555b1ba1bc6743a95d0149053ec1f8399a37399b/wikidataintegrator/wdi_core.py#L188-L199

I agree that this is confusing and should be removed, but since it creates a new item, this function might be used by WDI users.

andrawaag commented 5 years ago

item_name and domain parameter have been removed with version 0.2.0

andrawaag commented 5 years ago

addressed in: