search

uspdev / web-ldap-admin

https://gitlab.uspdigital.usp.br/uspdev/web-ldap-admin
3 stars 9 forks source link

Fix 128 - Mostrando a foto #133

Closed alecostaweb closed 2 years ago

alecostaweb commented 2 years ago

Fix #128 Require uspdev/wsfoto Parametrizado o uso do wsfoto Adicionada a informação no .env.example Adicionada string foto no controller LdapUser Exibindo a foto na view show-basic

thiagogomesverissimo commented 2 years ago

Uma dúvida, não seria melhor deixar esse recurso do wsfoto opcional podendo ser desativado no .env? Não sei se todos que usam o web-ldap-admin tem acesso ao wsfoto.

alecostaweb commented 2 years ago

E não está?

thiagogomesverissimo commented 2 years ago

No blade sim, mas no LdapUserController não.

Esse linha vai quebrar para quem não tem o Wsfoto:

$foto = ($codpes != null) ? \Uspdev\Wsfoto::obter($codpes) : '';

alecostaweb commented 2 years ago

$foto = (config('web-ldap-admin.mostrarFoto') == 1 and $codpes != null) ? \Uspdev\Wsfoto::obter($codpes) : '';

alecostaweb commented 2 years ago

Vou comitar assim então