bug en pagelet_comision.php ?

SIU-Guaraní SIU-Guaraní 2 - Técnicos
1

miren como está definido este método (v2.9.0):

public function get_data_reporte($pagina = 0, $todo = false){
$reg_por_pagina = $this->get_rpp();
$pagina = $this->get_pagina();
$minimo = ($pagina - 1) * $reg_por_pagina;
…

claramente no importa que página se pase por parámetro…

2

Hola Diego!!
El parámetro $pagina indica el número de página que se desea recuperar. Además, sirve para calcular la cantidad de registros por página.
En este caso, no se está utilizando el parámetro, pero la definición de la función es esa porque así lo indica la función abstracta ‘get_data_reporte’ del padre: pagelet_reporte (\siu\operaciones_comun\operaciones\reporte\pagelet_reporte).

3

mmm siempre que se sepa que el parámetro no es tenido en cuenta supongo que no es un bug :smiley:

preferiria quitar la línea “$pagina = $this->get_pagina();” y pasarle dicho valor en la llamada.

o detectar los casos en que el parámetro no se especificó en la llamada y en esos casos usar un valor por defecto:

public function get_data_reporte($pagina = null, $todo = false){
$reg_por_pagina = $this->get_rpp();
if (is_null($pagina)) {
$pagina = $this->get_pagina();
}
$minimo = ($pagina - 1) * $reg_por_pagina;
…

como sea, también aplica la regla de “todo lo que funciona no se toca” :wink: