Editions with wikidata, bookbrainz, and other new identifiers breaks save()

[mekarpeles]

[/usr/local/lib/python3.11/dist-packages/olclient/common.py](https://localhost:8080/#) in _validate_identifiers(identifiers)
     28         for id_type, values in identifiers.items():
     29             if id_type not in VALID_IDENTIFIERS:
---> 30                 raise AttributeError(
     31                     f"ID type '{id_type}' is not one of {VALID_IDENTIFIERS}"
     32                 )

AttributeError: ID type 'wikidata' is not one of ('olid', 'oclc', 'isbn_10', 'isbn_13', 'isbns', 'lccn', 'goodreads', 'librarything')

[mekarpeles]

Examples are ISBN: 9780394839127, 9780394839127 (wikidata), 9780394839127, 9780765381408 (bookbrainz)

https://github.com/internetarchive/openlibrary-client/blob/master/olclient/common.py#L8-L17

Maybe this should somehow pull from https://github.com/internetarchive/openlibrary/blob/a219065f0c68f06ea0144857e92972349734f9e3/openlibrary/plugins/openlibrary/config/edition/identifiers.yml#L196 (as we have lots of IDs not in this list besides wikidata)

[mekarpeles]

I believe my latest commit https://github.com/internetarchive/openlibrary-client/blob/master/olclient/common.py#L17-L18 may have solved this problem?

[D-1b-a]

[/usr/local/lib/python3.11/dist-packages/olclient/common.py](https://localhost:8080/#) in _validate_identifiers(identifiers)
     28         for id_type, values in identifiers.items():
     29             if id_type not in VALID_IDENTIFIERS:
---> 30                 raise AttributeError(
     31                     f"ID type '{id_type}' is not one of {VALID_IDENTIFIERS}"
     32                 )

AttributeError: ID type 'wikidata' is not one of ('olid', 'oclc', 'isbn_10', 'isbn_13', 'isbns', 'lccn', 'goodreads', 'librarything')

em português

O erro que você está enfrentando indica que o tipo de identificador "wikidata" não é reconhecido como um tipo válido pela função _validate_identifiers do pacote olclient. Especificamente, a mensagem é:

AttributeError: ID type 'wikidata' is not one of {VALID_IDENTIFIERS}

Isso quer dizer que o valor "wikidata" está sendo usado como um tipo de identificador, mas não está presente na lista ou conjunto VALID_IDENTIFIERS, que contém os tipos aceitos pelo código.

Para ajudar a resolver esse problema, algumas informações seriam importantes:

• Quais são os tipos de identificadores válidos definidos em VALID_IDENTIFIERS?
• Você tem certeza que o tipo "wikidata" deveria ser aceito nesse contexto?
• Qual operação ou função você está tentando executar que gerou esse erro? Isso pode ajudar a saber se o pacote precisa ser ajustado ou se o modo como você está passando o identificador está errado.

Se puder fornecer mais detalhes do contexto ou do trecho do código onde você passa esses identificadores, posso ajudar a encontrar uma forma de corrigir ou contornar o erro.

[D-1b-a]

Se "wikidata" já está nessa lista, o erro que você recebeu indicando que "wikidata" não é um identificador válido pode estar relacionado a um problema de formatação, talvez um erro de digitação, espaços em branco invisíveis, ou mesmo um problema no momento do envio do argumento.

Verifique:

Se a string passada como identificador é exatamente "wikidata", sem espaços ou quebras de linha extras.

Se o código que chama _validate_identifiers está usando a mesma variável ou valor corretamente.

Se o arquivo onde VALID_IDENTIFIERS está definido é exatamente o que está sendo carregado quando o erro ocorre (às vezes há versões diferentes do arquivo).

Se possível, mostre o trecho do código onde você chama a função que utiliza esses identificadores para que eu possa ajudar a identificar onde está o problema exato.> Examples are ISBN: 9780394839127, 9780394839127 (wikidata), 9780394839127, 9780765381408 (bookbrainz)

https://github.com/internetarchive/openlibrary-client/blob/master/olclient/common.py#L8-L17

Maybe this should somehow pull from https://github.com/internetarchive/openlibrary/blob/a219065f0c68f06ea0144857e92972349734f9e3/openlibrary/plugins/openlibrary/config/edition/identifiers.yml#L196 (as we have lots of IDs not in this list besides wikidata)