View Single Post
Old 11-21-2022, 10:06 AM   #3
KevinH
Sigil Developer
KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.
 
Posts: 8,933
Karma: 6361444
Join Date: Nov 2009
Device: many
Yes, mcontent being None here would cause an issue. I will test for that case and skip it if empty.

Passing in the sigil user interface language here would be a real pain as this parsing is being done in python here. I am not sure if the user interface language or default python locale lang would be a good bet either as books can be of any language really.

Not sure what value to put here. Perhaps a translated string: UNSPECIFIED or some obvious filler value should be used to force the user to actually specify a valid language. Or force it to default to en.

The spec is very clear and states there must be a value.

Last edited by KevinH; 11-21-2022 at 10:45 AM.
KevinH is offline   Reply With Quote