Skip to content

Commit

Permalink
fix type
Browse files Browse the repository at this point in the history
  • Loading branch information
thiagogomesverissimo committed Jun 29, 2023
1 parent 484bb31 commit 8a39289
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 28 deletions.
2 changes: 1 addition & 1 deletion app/Jobs/SincronizaReplicado.php
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ public function sync($pessoas)

foreach ($pessoas as $pessoa) {
// Chama método para criar ou atualizar passando o array da pessoa
LdapUser::criarOuAtulizarPorArray($pessoa);
LdapUser::criarOuAtualizarPorArray($pessoa);
}
}
}
Expand Down
2 changes: 1 addition & 1 deletion app/Ldap/User.php
Original file line number Diff line number Diff line change
Expand Up @@ -362,7 +362,7 @@ public static function desativarUsers($desligados)
* @param array $pessoa
* @author Alessandro Costa de Oliveira 11/03/2022
*/
public static function criarOuAtulizarPorArray($pessoa, $metodo = '')
public static function criarOuAtualizarPorArray($pessoa, $metodo = '')
{
// setando username e codpes (similar loginListener)
switch (strtolower(config('web-ldap-admin.campoCodpes'))) {
Expand Down
28 changes: 2 additions & 26 deletions app/Listeners/LoginListener.php
Original file line number Diff line number Diff line change
Expand Up @@ -59,31 +59,6 @@ public function handle(Login $event)
// TODO completar os valores necessários quando a pessoa não tem vínculo, mas pode logar
if (config('web-ldap-admin.sincLdapLogin') == 1) {
if (!isset($pessoa)) {
// Com vínculo ativo ('ALUNOGR', 'ALUNOPOS', 'ALUNOCEU', 'ALUNOEAD', 'ALUNOPD', 'ALUNOCONVENIOINT', 'SERVIDOR', 'ESTAGIARIORH')
// TODO precisa melhorar a criação do array pessoa para chamar o método para criar ou atualizar
// Principalmente se a pessoa for Servidor e também Alunode Graduação, Aluno de Pós-Graduação ou outro vínculo dos mencionados acima


// Não entendi porque aqui estava sendo carregado todas pessoas do mesmo grupo
// por exemplo: quando um aluno de graduação tenta logar, esse array é carregado com
// milhares de alunos dependendo da unidade, tornando o login muito lento

// Tem um erro também com a variável $vinculoPessoa que é criado dentro de um if
// mas é chamada fora, e o sistema quebra quando o fluxo não entra no if
// Vou deixar comentado até conversar com masaki e alessandro para otimizarmos esse login
// 12/04/2022 - @thiagogomesverissimo

/*
$tiposVinculos = Pessoa::tiposVinculos(config('web-ldap-admin.replicado_unidade'));
foreach ($vinculos as $vinculo) {
if (array_search($vinculo, array_column($tiposVinculos, 'tipvinext'))) {
$vinculoPessoa = $vinculo;
}
}
$pessoas = Pessoa::ativosVinculo($vinculoPessoa, config('web-ldap-admin.replicado_unidade'));
$pessoa = array_search($event->user->username, array_column($pessoas, 'codpes'));
$pessoa = $pessoas[$pessoa];
*/

$pessoa = Pessoa::dump($event->user->codpes);
$pessoa['codema'] = $event->user->email;
Expand All @@ -98,7 +73,8 @@ public function handle(Login $event)
}

// Chama método para criar ou atualizar passando o array da pessoa
LdapUser::criarOuAtulizarPorArray($pessoa);

LdapUser::criarOuAtualizarPorArray($pessoa);

Session::flash('alert-success', 'Informações sincronizadas com Sistemas Corporativos');
}
Expand Down

0 comments on commit 8a39289

Please sign in to comment.