Torna Alguns Campos apenas editáveis por superusuários #633
Conversation
- Adiciona Js para desabilitar botoes de remover e adicionar objetos em InlinePanel
- Adiciona __str__ em BaseInstitution
- Adiciona edit_handler em ScieloJournal - remove classname e adiciona read_only em Annotation.panels
| @@ -4,12 +4,6 @@ | |||
| --color-primary-dark: #4c77d5; | |||
| } | |||
|
|
|||
| .myReadonlyInput input[type="text"]{ | |||
There was a problem hiding this comment.
@samuelveigarangel em refatoração, procure inserir coisas novas primeiro, e somente no final remover
| @@ -2,7 +2,9 @@ | |||
| document.addEventListener('DOMContentLoaded', function() { | |||
| const collectionSelect = document.querySelector('select[name="collection"]'); | |||
| const journalSelect = document.querySelector('select[name="journal"]'); | |||
| const currentJournalId = journalSelect.value; // Armazena o valor atual do journal | |||
| if (journalSelect) { | |||
| const currentJournalId = journalSelect.value; // Armazena o valor atual do journal | |||
There was a problem hiding this comment.
@samuelveigarangel é ok criar a variável condicionada? mais adiante não ocorrerá erro de uso de variável sem definição?
| readonly_fields = [] | ||
|
|
||
| def get_edit_handler(self): | ||
| edit_handler = super().get_edit_handler() |
There was a problem hiding this comment.
@samuelveigarangel a lógica então é deixar tudo fechado e abrir só para os superusers?
| @@ -31,7 +31,7 @@ class OfficialJournalEditView(BaseEditView): | |||
| "iso_short_title", | |||
| "issn_print", | |||
| "issn_electronic", | |||
| "issnl" | |||
| "issnl", | |||
There was a problem hiding this comment.
@samuelveigarangel não estou convencida da abordagem para os dados dos periódicos e relacionados, pois estes dados devem ser de responsabilidade das coleções e dos editores. Se deixamos com readonly como eles poderão alterar? Aí perde-se a autonomia, vai centralizar nos administradores de sistema.
O que tem que ser feito é assegurar que cada membro da coleção só pode atuar nos registros da sua coleção. E cada membro da equipe editorial só pode atuar no seu periódico.
|
@robertatakenaka Entendo. Porém, conforme em: #548, certos campos devem ser alterados apenas por admin SciELO? É o que essa PR faz, torna alguns campos editáveis apenas por admin SciELO (superuser). |
|
Isso que você comentou, já foi feito em: #562 |
|
|
||
| # Name of Groups | ||
| ADMIN_SCIELO = "Administrator SciELO" |
There was a problem hiding this comment.
@samuelveigarangel os tipos de usuários deveriam ficar juntos para facilitar manutenção
There was a problem hiding this comment.
67f6479
Mantem todos os nomes juntos.
Deixei dessa forma para nao dar conflito com essa pr
O que esse PR faz?
Utilizando a classe 'BaseEditView', é possível personalizar certos campos dependendo da condição de superusuário do usuário. A classe 'BaseEditView' sobrescreve o método 'get_edit_handler' da superclasse para implementar essas customizações.
Onde a revisão poderia começar?
Pelo commit a44a5bd
Como este poderia ser testado manualmente?
Algum cenário de contexto que queira dar?
N/A
Screenshots
Quais são tickets relevantes?
#621
#620
#619
#618
#617
#615
#548
Referências
N/A