Page 1 of 3
Report bugs for Version 1.2.4 here
Posted: Sat Nov 20, 2010 10:28 am
by Brent
Report any bugs for Version 1.2.4
Re: Report bugs for Version 1.2.4 here
Posted: Tue Nov 23, 2010 10:53 am
by gob33
1] At install, i have this warning (non bloquant) at step 2
Code: Select all
Data too long for column 'symbol_left' at row 1
Data too long for column 'symbol_left' at row 1
Data too long for column 'symbol_left' at row 1
...
Data too long for column 'title' at row 1
Data too long for column 'title' at row 1
...
Comes from upgrade.sql
2] The notice
Code: Select all
Support AlegroCart and AlegroCart will Support you.
©2010 AlegroCart Powered
Developed by: Your Developer Team
has a font too big for me and should be more discrete.
You can also put your name in the Copyright to be more concrete.
3] ALEGROCART-1.2.4/upload/admin/?controller=image_display&action=insert
Settings are the same on each langage page
4] ALEGROCART-1.2.4\upload\admin\language\french\controller\currency.php
$_['error_default'] is twice
5] Centering the title of Welcome page module
6] I dislike the new 123 icon at beginning of lines in admin.
This is not explicit to a newcommer to understand he must click on it to access under.
The directory icon of v1.2.3 was better.
e.g: To entry taxes, not evidence to click on a line for a newcommer.
7] Zones for a country should be accessed by clicking on an icon on the country line.
Its easier and faster to do like that than browsing all zones of the world and enabling/disabling.
Re: Report bugs for Version 1.2.4 here
Posted: Tue Nov 23, 2010 2:30 pm
by Brent
1] At install, i have this warning (non bloquant) at step 2
Will check to SQL. I have never got this error even with full reporting.
3] ALEGROCART-1.2.4/upload/admin/?controller=image_display&action=insert
Settings are the same on each langage page
What settings are your referring to.
Code: Select all
4] ALEGROCART-1.2.4\upload\admin\language\french\controller\currency.php
$_['error_default'] is twice
Yes, that should be error_disable
5] Centering the title of Welcome page module
It is centered on every browser except IE6. Haven't had time to work on 6.
7] Zones for a country should be accessed by clicking on an icon on the country line.
Its easier and faster to do like that than browsing all zones of the world and enabling/disabling.
I think we had this discussion before.
First, you only need to disable a country to make the zones not available.
If you want to disable on a few zones for a country, the sorting is by country, then by zone, so all zones are grouped with their parent country.
If you enter the country in the search box, you will have only that country's zones listed.
Re: Report bugs for Version 1.2.4 here
Posted: Tue Nov 23, 2010 4:28 pm
by gob33
2] Copyright is legaly important. Who own the copyright ?
5] Only that title is un-centered on my IE6. All others are good. Missing an align="center" somewhere ?
7] Already discussed; i agree.
8] ALEGROCART-1.2.4/upload/?controller=account_invoice&order_id=9
In Store front, Order history for an account displays the status of an order in each language installed.
So i have the same order status in english and french (2 lines)
9] ALEGROCART-1.2.4/upload/admin/?controller=order&order_id=9&action=update
In Admin, details for an order shows asterix (amount with taxes) in columns Unit price, extended amount,net cost. Should be without taxes (no asterix).
10] ALEGROCART-1.2.4/upload/admin/?controller=customer
In Admin, customer details does not permit to see all customer addresses (only the default one)
Re: Report bugs for Version 1.2.4 here
Posted: Tue Nov 23, 2010 5:01 pm
by Brent
2] Copyright is legaly important. Who own the copyright ?
I own the copyright. It will remain that way because I have heard discussions on other carts about possibly charging for use.
It was started as opensource and will remain that way, free of charge to users and developers.
9] ALEGROCART-1.2.4/upload/admin/?controller=order&order_id=9&action=update
In Admin, details for an order shows asterix (amount with taxes) in columns Unit price, extended amount,net cost. Should be without taxes (no asterix).
That change was finished the day before release. We will be making a few adjustments in the next while.
10] ALEGROCART-1.2.4/upload/admin/?controller=customer
In Admin, customer details does not permit to see all customer addresses (only the default one)
That is by design. We only showed the main address for reference. It is not editable. It was decided that customer privacy dictated that we did not require more information to be available.
Re: Report bugs for Version 1.2.4 here
Posted: Wed Nov 24, 2010 9:04 am
by gob33
5] ALEGROCART-1.2.4\upload\catalog\template\default\module\homepage.tpl
Line 8: <div class="headingpadded"><div class="center"><h1><?php echo $heading_title; ?></h1></div>
Removing <div class="center"> and the title is centered.
11] ALEGROCART-1.2.4\upload\admin\controller\report_viewed.php
Bar graph is always at 100% for every product when in French (ok in English)
Comes from the 'round' instruction which place a comma in french and a point in english in numbers for decimals. Change to:
Code: Select all
//$percent=$total?round(($result['viewed'] / $total) * 100, 2):0;
$percent=$total?number_format(($result['viewed'] / $total) * 100, 2, '.', ''):0;
Re: Report bugs for Version 1.2.4 here
Posted: Wed Nov 24, 2010 1:17 pm
by Brent
11] ALEGROCART-1.2.4\upload\admin\controller\report_viewed.php
Bar graph is always at 100% for every product when in French (ok in English)
Comes from the 'round' instruction which place a comma in french and a point in english in numbers for decimals. Change to:
Code: Select all
//$percent=$total?round(($result['viewed'] / $total) * 100, 2):0;
$percent=$total?number_format(($result['viewed'] / $total) * 100, 2, '.', ''):0;
I'm still trying to get my head around this one.
I checked the php site on round and it makes no mention of language and specifically says do not include formatting.
Why would it place a comma in french? I am confused. It appears it should be decimal point by the PHP manual.
Is this some special setup.
Re: Report bugs for Version 1.2.4 here
Posted: Wed Nov 24, 2010 4:52 pm
by gob33
11] Do know why exactly but if you display the source of the page from browser, there is a comma in "width=47,5%" instead of "width=47.5%". Im on XP, locale settings comes from Windows. And ive never had bar graph correct in previous versions when in french. Perhaps, the conversion from float to string do that.
Re: Report bugs for Version 1.2.4 here
Posted: Wed Nov 24, 2010 4:59 pm
by Brent
I found out localization does change rounding as well as other things.
I would try to set localization to somewhere else and see if it changes.
Rounding is an integral part of the cart. The new balanced invoice required all calculations to be rounded.
Re: Report bugs for Version 1.2.4 here
Posted: Thu Nov 25, 2010 10:15 am
by Brent
It appears round will return comma in rounding function, but will not accept the comma for input.
Another problem is a mix of commas and decimal point in database currency items.
This can be set. On an XP System, go to control panel, Regional and Language Options.
You will select tab Regional Option, and customize.
You can change the currency defaults here.
The French setting allows a choice of Comma or Decimal. Select Decimal.
The cart automatically changes the formatting based on language selected.
An alternative is to insert setlocale(LC_ALL,"En-Us") or setlocale(LC_NUMERIC, "En-Us") in common.php at the top.
This might require more setup and is only active in the current script.