ArthurTaylor added a comment.

  Had a quick look at this today. It's pretty easy to reproduce the bug with 
just the library code on its own - see this fork/branch on Github 
<https://github.com/DataValues/Geo/compare/master...codders:DataValues-Geo:bugfix/type-checking-formatter-options?expand=1>.
  
  My suggested approach would be first to make the libraries themselves robust 
and prevent them throwing type errors. To do this, I would deprecate the 
generic setOption and getOption functions, and replace them with 
{get,set}{String,Int,Bool}Option functions that are robust to bad input. Then 
we can catch InvalidFormatterOptions in our API code and do something more 
graceful there. Would be happy to take a look at this.

TASK DETAIL
  https://phabricator.wikimedia.org/T323778

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: ArthurTaylor
Cc: ArthurTaylor, ItamarWMDE, Michael, Lucas_Werkmeister_WMDE, Aklapper, 
Danny_Benjafield_WMDE, Astuthiodit_1, karapayneWMDE, Invadibot, maantietaja, 
Akuckartz, Nandana, Lahi, Gq86, GoranSMilovanovic, QZanden, KimKelting, 
LawExplorer, _jensen, rosalieper, Scott_WUaS, Wikidata-bugs, aude, Mbch331
_______________________________________________
Wikidata-bugs mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to