Update rodape.php#8
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the DataTables initialization in the footer view to include a dynamic page length setting based on the configuration. The review feedback suggests improving the robustness of this change by providing a default value and ensuring the value is cast to an integer to avoid potential JavaScript syntax errors if the configuration is missing.
| var dataTableEnabled = '<?= $configuration['control_datatable'] ?>'; | ||
| if(dataTableEnabled == '1') { | ||
| $('#tabela').dataTable( { | ||
| "pageLength": <?= $configuration['per_page'] ?>, |
There was a problem hiding this comment.
O código atual pode causar um erro de sintaxe JavaScript se a configuração per_page não estiver definida, o que pode quebrar a funcionalidade das tabelas. Para tornar o código mais robusto, é recomendável fornecer um valor padrão caso a configuração não exista.
Além disso, a propriedade pageLength do DataTables espera um número inteiro. Converter o valor para int garante que o tipo de dado correto seja sempre utilizado.
"pageLength": <?= (int) ($configuration['per_page'] ?? 10) ?>,1 participant
Adicionar a propriedade pageLength correspondente ao valor configurado no sistema.
Ao configurar a quantidade de registros exibidos por página, quando a visualização em DataTables está ativada e configurada para exibir acima de 10 registros por página, a exibição de quantidade de registros por página nas telas de listagem não correspondem ao valor configurado.