Warning: Division by zero in

[Lazard]

Aunque la funcionalidad es correcta , me sale este error, se lo que significa, pero no se como sacarlo.

error -> Warning: Division by zero in /home/7384732984732948/public_html/wp-content/plugins/bp-album/includes/bp-album-templatetags.php on line 80

Código PHP:

   
class BP_Album_Template {
    
    var $current_picture = -1;
    var $picture_count = 0;
    var $pictures;
    var $picture;
 
    var $in_the_loop;
 
    var $pag_page;
    var $pag_per_page;
    var $pag_links;
 
    function BP_Album_Template( $args = '' ) {
        $this->__construct( $args);
    }
    
    function __construct( $args = '' ) {
        global $bp;
 
        
        $defaults = bp_album_default_query_args();
        
        $r = apply_filters('bp_album_template_args',wp_parse_args( $args, $defaults ));
        extract( $r , EXTR_SKIP);
 
        $this->pag_page = $page;
        $this->pag_per_page = $per_page;
        $this->owner_id = $owner_id;
        $this->privacy= $privacy;
        
        /***
         * You can use the "type" variable to fetch different things to output.
         * For example on the groups template loop, you can fetch groups by "newest", "active", "alphabetical"
         * and more. This would be the "type". You can then call different functions to fetch those
         * different results.
         */
 
        $total = bp_album_get_picture_count($r);
        $this->pictures = bp_album_get_pictures($r);
 
        // Item Requests
        if ( !$max || $max >= $total )
            $this->total_picture_count = $total;
        else
            $this->total_picture_count = $max;
 
        if ( !$max || $max >= count($this->pictures))
            $this->picture_count = count($this->pictures);
        else
            $this->picture_count = $max;
 
        $this->pag_links = paginate_links( array(
            'base' => $bp->displayed_user->domain . $bp->album->slug .'/'. $bp->album->pictures_slug .'/%_%',
            'format' => '%#%',
            'total' => ceil( (int) $this->total_picture_count / (int) $this->pag_per_page ), /// es aqui donde el error !
            'current' => (int) $this->pag_page,
            'prev_text' => '←',
            'next_text' => '→',
            'mid_size' => 1
        ));
        
 
        if ($this->picture_count)
            $this->picture = $this->pictures[0];
        
    } 
   

He probado con esto pero no funciona

if(empty($this))
{
echo "M";
}
else
{
echo "O";
}

[abimaelrc]

Dice que división por 0 no es valida (lo cual es lógico), lo único que tienes que hacer es verificar si tiene como valor 0 y si es así muestras una cosa y si no pues haces la división

[Lazard]

Iniciado por abimaelrc

Dice que división por 0 no es valida (lo cual es lógico), lo único que tienes que hacer es verificar si tiene como valor 0 y si es así muestras una cosa y si no pues haces la división

Pero al ser un array no entiendo muy bien como hacerlo :( esta claro que me estas diciendo que verifique si la variable tiene el valor 0 con un if, si fuera zero pues le pondria un echo"loquesea" y si no es zero que continue con el codigo, pero al ser un array no se que variable tengo que cojer o_O

[Nexus10]

Hola Lazard

La variable pag_per_page es la que está dando el error. ¿A que se refiere esa variable?, ¿a páginas?. Si se refiere a páginas, no creo que un album pueda tener cero páginas. Tendrá por lo menos una, así que esa variable no tendría por qué ser cero. Verifica que esté tomando valores correctos, o donde definas la variable asegúrate que no puede ser cero nunca.

[Lazard]

Iniciado por Nexus10

Hola Lazard

La variable pag_per_page es la que está dando el error. ¿A que se refiere esa variable?, ¿a páginas?. Si se refiere a páginas, no creo que un album pueda tener cero páginas. Tendrá por lo menos una, así que esa variable no tendría por qué ser cero. Verifica que esté tomando valores correctos, o donde definas la variable asegúrate que no puede ser cero nunca.

Entiendo, hmmm sigue igual después de haberle introducido el siguiente codigo para arreglar el valor inicial de la variable pag_per_page

Código PHP:

          $this->pag_links = paginate_links( array(
            'base' => $bp->displayed_user->domain . $bp->album->slug .'/'. $bp->album->pictures_slug .'/%_%',
            'format' => '%#%',
            'total' => ceil( (int) $this->total_picture_count / (int) $this->pag_per_page ),
            'current' => (int) $this->pag_page,
            'prev_text' => '←',
            'next_text' => '→',
            'mid_size' => 1
        ));
        
        if ($pag_per_page = '0')
        {
            $pag_per_page ++;
        } 
   

[masterpuppet]

Estas asignando el valor en el if no comparando, debería ser:

Código PHP:

if ($pag_per_page == 0) {
    $pag_per_page ++;
}
 

Saludos.

[Nexus10]

Iniciado por Lazard

Entiendo, hmmm sigue igual después de haberle introducido el siguiente codigo para arreglar el valor inicial de la variable pag_per_page

Código PHP:

          $this->pag_links = paginate_links( array(
            'base' => $bp->displayed_user->domain . $bp->album->slug .'/'. $bp->album->pictures_slug .'/%_%',
            'format' => '%#%',
            'total' => ceil( (int) $this->total_picture_count / (int) $this->pag_per_page ),
            'current' => (int) $this->pag_page,
            'prev_text' => '←',
            'next_text' => '→',
            'mid_size' => 1
        ));
        
        if ($pag_per_page = '0')
        {
            $pag_per_page ++;
        } 
   

No creo que sea ahí donde debas poner ese if porque el array ya está construído, y si la variable valía cero te seguirá dando error.
Yo no estoy muy puesto en programación orientada a objetos, pero entiendo que en alguna parte de tu programa estarás usando la clase BP_Album_Template. Cuando creas un objeto de esa clase y le pasas los parámentros al constructor, es ahí cuando tienes que asegurarte de que el valor de $pag_per_page que le pasas es distinto de cero.

Puede que sea mejor que preguntes en el subforo de PHP OO, quizá ahí puedan orientarte mejor sobre tu problema.