Report bugs for Version 1.2.4 here

Enter AlegroCart Bugs Here, but if you're not certain, post in General support first.
User avatar
Brent
Site Admin
Posts: 4459
Joined: Sat Dec 12, 2009 3:35 pm
Location: Canada eh

Report bugs for Version 1.2.4 here

Post by Brent » Sat Nov 20, 2010 10:28 am

Report any bugs for Version 1.2.4

gob33
Posts: 107
Joined: Wed Apr 07, 2010 3:10 pm

Re: Report bugs for Version 1.2.4 here

Post by gob33 » Tue Nov 23, 2010 10:53 am

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.

User avatar
Brent
Site Admin
Posts: 4459
Joined: Sat Dec 12, 2009 3:35 pm
Location: Canada eh

Re: Report bugs for Version 1.2.4 here

Post by Brent » Tue Nov 23, 2010 2:30 pm

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.

gob33
Posts: 107
Joined: Wed Apr 07, 2010 3:10 pm

Re: Report bugs for Version 1.2.4 here

Post by gob33 » Tue Nov 23, 2010 4:28 pm

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)

User avatar
Brent
Site Admin
Posts: 4459
Joined: Sat Dec 12, 2009 3:35 pm
Location: Canada eh

Re: Report bugs for Version 1.2.4 here

Post by Brent » Tue Nov 23, 2010 5:01 pm

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.

gob33
Posts: 107
Joined: Wed Apr 07, 2010 3:10 pm

Re: Report bugs for Version 1.2.4 here

Post by gob33 » Wed Nov 24, 2010 9:04 am

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;

User avatar
Brent
Site Admin
Posts: 4459
Joined: Sat Dec 12, 2009 3:35 pm
Location: Canada eh

Re: Report bugs for Version 1.2.4 here

Post by Brent » Wed Nov 24, 2010 1:17 pm

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.

gob33
Posts: 107
Joined: Wed Apr 07, 2010 3:10 pm

Re: Report bugs for Version 1.2.4 here

Post by gob33 » Wed Nov 24, 2010 4:52 pm

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.
Last edited by gob33 on Wed Nov 24, 2010 5:02 pm, edited 2 times in total.

User avatar
Brent
Site Admin
Posts: 4459
Joined: Sat Dec 12, 2009 3:35 pm
Location: Canada eh

Re: Report bugs for Version 1.2.4 here

Post by Brent » Wed Nov 24, 2010 4:59 pm

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.

User avatar
Brent
Site Admin
Posts: 4459
Joined: Sat Dec 12, 2009 3:35 pm
Location: Canada eh

Re: Report bugs for Version 1.2.4 here

Post by Brent » Thu Nov 25, 2010 10:15 am

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.

Locked