Skip to content

Classe TokenAuthorization refatorada#4

Open
isaacmsl wants to merge 1 commit intoifrn-oficial:masterfrom
isaacmsl:patch-token-refactor
Open

Classe TokenAuthorization refatorada#4
isaacmsl wants to merge 1 commit intoifrn-oficial:masterfrom
isaacmsl:patch-token-refactor

Conversation

@isaacmsl
Copy link
Copy Markdown

@isaacmsl isaacmsl commented Oct 1, 2020

  • Corrigida nomenclatura de parâmetro na documentação da classe TokenAuthorization ("token" -> "value");
  • Adicionado construtor para a classe TokenAuthorization;
  • Adicionados atributos com marcadores "this";

Alguns detalhes simples de refatoração, como em:

this.isValid = function() {
if (Cookies.get('suapToken') && value != null) {
return true;
}
return false;
};

Que foi transformado em:

  isValid = function () {
    return Cookies.get('suapToken') && this.value;
  };

Para versões futuras, acredito que as classes devem ser separadas em módulos Javascript.

@adorilson
Copy link
Copy Markdown

@paivatulio Poderia analisar esse PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants