MediaWiki Discussão:Mobile.js

Includes

@!Silent e He7d3r: Poderiam analisar essa edição? Aparentemente deveríamos usar alguma função para substituir o .includes, mas a tentativa do Jon gerou um erro e não sei como fazer nesse caso. ━ ALBERTOLEONCIO Who, me? 04h45min de 18 de abril de 2021 (UTC)

@Albertoleoncio, @!Silent: Acredito ter sido apenas um probleminha de lógica. Confiram se minha edição faz sentido, por favor. Helder 10h10min de 18 de abril de 2021 (UTC)

@He7d3r Pra mim está tudo certo. Só nunca tinha visto esse uso da mw.config.get com aquele segundo parâmetro ([]); qual a diferença? !Silent (discussão) 18h43min de 18 de abril de 2021 (UTC)

Ótimo!

Sobre o .get: pelo código-fonte, o segundo parâmetro permite definir um valor padrão a ser usado caso não exista a chave especificada no primeiro. Então se por algum motivo wgUserGroups não existir na config, será retornada uma array vazia. Talvez não seja necessário nesse caso específico (não sei se wgUserGroups deixa de existir em algum caso), mas como o Jon tem corrigido esse tipo de coisa em massa em todas as wikis, a precaução é justificável (e o pequeno acidente também...) Helder 18h58min de 18 de abril de 2021 (UTC)