Division by Zero on com_virtuemart/models/product.php [SOLVED]

[inopia]

Hi dudes,

I´ve recently upgrade tu 2.0.8a version. Aparently everything runs ok but system show this warning:

Warning: Division by zero in */administrator/components/com_virtuemart/models/product.php on line 516

How can i do to fix this? It´s a bug from last version 2.0.8a?

Thank you.

[inopia]

I´ve observed if select 6 items per page or more this warning dissapears and when i select again 3 items per page all works but if you reloads the site with clean cache this issue appears again.

The warning appears when this line are evaluated:

$rest = $suglimit%$category->products_per_row;

I think $suglimit gets my item limit per row from my VirtueMart settings (10). by the way a division by zero isn´t possible but...

Suggestions?

Thxu

[bytelord]

Have the same issue with v2.0.8a.

As i see the $category->products_per_row does not return the products per row, but zero, for that reason that error appears.

Its a bug i suppose that need to get fixed.

Nikos

[bytelord]

i find out what have been changed.

the $category->products_per_row is coming from the category configuration on the backend. So by default that value is '0'.
That means that category oprions from shop->configuration are overided from category products_per_row.

If you go to each category and place there an option for products per row then the error is resolved.

[inopia]

bytelord, you´re right!

I´ve solved it like you said. I´m a beginner on VM and i don´t think could be this possibility.

If you need something i will try to help you.

Thank you very much.

[bytelord]

You are welcome. Its a bug as i have seen until now, except if there is something wrong with the upgrade from 2.06 to 2.0.8a.

Until now what i have seen is that the "Default number of products in a row" under Configuration->Templates->Shopfront settings is been overrided from the category configuration.

[inopia]

"It's not a bug; it's an undocumented feature!" LOL

[vynette]

Thank you so much Bytelord. I have only just started building my site a few days ago for the first time. I thought I did something wrong. I have changed my 42 categories now and all is good thank you. When they upgrade again next time do they fix that ? And then will the configuration/template numbers over ride those ones I just changed  if I want to change them or will I have to go back  in and change all of them all again ? thank you for your help.

[inopia]

Seriously.

Probably we must set up all the fields correctly and probably the devs must handle default field values when users generated categories, yet users have to know that zero elements in a row have no sense.

[esanchez1091]

Thanks bytelord that was the fix.....


bytelord

    Beginner
    *
    Posts: 6
        View Profile
        Personal Message (Online)

Re: Division by Zero on com_virtuemart/models/product.php
« Reply #3 on: Today at 06:41:54 »

    Quote

i find out what have been changed.

the $category->products_per_row is coming from the category configuration on the backend. So by default that value is '0'.
That means that category oprions from shop->configuration are overided from category products_per_row.

If you go to each category and place there an option for products per row then the error is resolved.


[attachment cleanup by admin]

[Milbo]

It should be even enough to write this into the vm config. But I checked the code again and found that I am not checking for this, please use this code

Code: [Select]

if(empty($limit)){
if(!empty($category->limit_list_initial)){
$suglimit = $category->limit_list_initial;
} else {
if(empty($category->limit_list_step)){
$suglimit = VmConfig::get ('list_limit', 20);
} else {
$suglimit = $category->limit_list_step;
}
}
if(empty($category->products_per_row)){
$category->products_per_row = VmConfig::get ('products_per_row', 3);
}
$rest = $suglimit%$category->products_per_row;
$limit = $suglimit - $rest;

}
Then it should work always and use the standard value.

[bytelord]

Thank you for the code snippet.

[inopia]

Thank you for the code.

[cecilia2330]

It should be even enough to write this into the vm config. But I checked the code again and found that I am not checking for this, please use this code

Code: [Select]

if(empty($limit)){
if(!empty($category->limit_list_initial)){
$suglimit = $category->limit_list_initial;
} else {
if(empty($category->limit_list_step)){
$suglimit = VmConfig::get ('list_limit', 20);
} else {
$suglimit = $category->limit_list_step;
}
}
if(empty($category->products_per_row)){
$category->products_per_row = VmConfig::get ('products_per_row', 3);
}
$rest = $suglimit%$category->products_per_row;
$limit = $suglimit - $rest;

}
Then it should work always and use the standard value.

Sorry that, anyone know where to find the vm config. ??
I want to try Milbo 's solution

[Loris]

It should be even enough to write this into the vm config. But I checked the code again and found that I am not checking for this, please use this code

Code: [Select]

if(empty($limit)){
if(!empty($category->limit_list_initial)){
$suglimit = $category->limit_list_initial;
} else {
if(empty($category->limit_list_step)){
$suglimit = VmConfig::get ('list_limit', 20);
} else {
$suglimit = $category->limit_list_step;
}
}
if(empty($category->products_per_row)){
$category->products_per_row = VmConfig::get ('products_per_row', 3);
}
$rest = $suglimit%$category->products_per_row;
$limit = $suglimit - $rest;

}
Then it should work always and use the standard value.

Sorry that, anyone know where to find the vm config. ??
I want to try Milbo 's solution

in /administrator/components/com_virtuemart/models/product.php

replace the original 2.0.8a code

Code: [Select]

if(empty($limit)){
if(!empty($category->limit_list_initial)){
$suglimit = $category->limit_list_initial;
} else {
if(empty($category->limit_list_step)){
$suglimit = VmConfig::get ('list_limit', 20);
} else {
$suglimit = $category->limit_list_step;
}
}

$rest = $suglimit%$category->products_per_row;
$limit = $suglimit - $rest;

}
with the Milbo's code