Post by i2Paq » Mon Dec 06, 2010 6:25 am

Qphoria wrote:I had asked him to release it as a mod first to get more exposure and testing, then I would add it to the next release.. but he never did so I haven't released anything
I've PM'ed you the files.

I will make a "official" release of these files tomorrow with a explanation of the limitations: no selectable shipment method, no recognition of Quantity discount etc.

Norman in 't Veldt
Moderator OpenCart Forums

_________________ READ and Search BEFORE POSTING _________________

Our FREE search: Find your answer FAST!.

[How to] BTW + Verzend + betaal setup.


User avatar
Global Moderator

Posts

Joined
Mon Nov 09, 2009 7:00 pm
Location - Winkel - The Netherlands

Post by opencart 99 » Fri Dec 10, 2010 11:10 pm

hi
When I try to send email to my customers show me this message :
/admin/controller/sale/contact.php on line 280

New member

Posts

Joined
Mon May 03, 2010 6:16 am

Post by hawkey » Fri Dec 10, 2010 11:15 pm

Qphoria wrote:I had asked him to release it as a mod first to get more exposure and testing, then I would add it to the next release.. but he never did so I haven't released anything
I am sorry, I work on another mods. I will do it now with i2paq's description.

Visit OpencartEx - Opencart extensions
Opencart rady a návody česky | Podpora Opencart


Active Member

Posts

Joined
Sun Apr 25, 2010 12:10 am
Location - Olomouc, Czech Republic, Europe

Post by Xsecrets » Sat Dec 11, 2010 1:27 am

hawkey wrote:
Qphoria wrote:I had asked him to release it as a mod first to get more exposure and testing, then I would add it to the next release.. but he never did so I haven't released anything
I am sorry, I work on another mods. I will do it now with i2paq's description.
I think ipaq already posted it.

OpenCart commercial mods and development http://spotonsolutions.net
Layered Navigation
Shipment Tracking
Vehicle Year/Make/Model Filter


Guru Member

Posts

Joined
Sun Oct 25, 2009 3:51 am
Location - FL US

Post by i2Paq » Sat Dec 11, 2010 2:57 am

Xsecrets wrote:
hawkey wrote:
Qphoria wrote:I had asked him to release it as a mod first to get more exposure and testing, then I would add it to the next release.. but he never did so I haven't released anything
I am sorry, I work on another mods. I will do it now with i2paq's description.
I think ipaq already posted it.

Yes: Partial fix for the Order Edit in 1.4.9.1 and 1.4.9.2

Norman in 't Veldt
Moderator OpenCart Forums

_________________ READ and Search BEFORE POSTING _________________

Our FREE search: Find your answer FAST!.

[How to] BTW + Verzend + betaal setup.


User avatar
Global Moderator

Posts

Joined
Mon Nov 09, 2009 7:00 pm
Location - Winkel - The Netherlands

Post by Johnathan » Sat Dec 11, 2010 3:38 am

A very tiny bug that I assume affects all versions, though I've only confirmed it on 1.4.9.2:

If the customer's first or last name has a space in it, it will put the names on different lines in the invoice. For example, if the customer enters their first name as "John " and their last name as "Doe", the invoice will show:

John
Doe
123 Address St.
etc. etc.

Image Image Image Image Image


User avatar
Administrator

Posts

Joined
Fri Dec 18, 2009 3:08 am


Post by hawkey » Sat Dec 11, 2010 5:52 am

I know. I released it as a free mod in Extensions.
i2Paq wrote:
Xsecrets wrote: I think ipaq already posted it.

Yes: Partial fix for the Order Edit in 1.4.9.1 and 1.4.9.2

Visit OpencartEx - Opencart extensions
Opencart rady a návody česky | Podpora Opencart


Active Member

Posts

Joined
Sun Apr 25, 2010 12:10 am
Location - Olomouc, Czech Republic, Europe

Post by rogery » Sat Dec 11, 2010 10:51 pm

v1.4.9.2

I just found out that if I remove discounted items from an order in Admin, the order discount amount remains the same 10% of the original order and doesn't adjust it for removed items.

Does anyone know how to fix it?

Roger

New member

Posts

Joined
Wed May 26, 2010 5:55 pm

Post by Qphoria » Sun Dec 12, 2010 2:50 pm

OK... While waiting for 1.5.0, I'm going to release a 1.4.9.3 FULL and PATCH.

Here are the changes. Have I missed anything:
== OpenCart v1.4.9.3 ==

FIXES:
- BUG: Notice: Undefined index: postcode_required if country set to please select
- BUG: customer checks out and wants to change either the Shipping or Payment address, a "The page you requested cannot be found" error appears
- BUG: Homepage modules cannot use document library class methods because the "header" module was loading before the homepage modules.
- BUG: Disabling Bestseller items causes the limit to be (limit-1)
- BUG: setting controller had copy/paste bug for icon/logo check
- BUG: php.ini setting "max_upload_filesize" should be "upload_max_filesize" which explains why it didn't work
- BUG: Paypal standard sandbox mode shows "text_testmode" instead of the correct testmode message
- BUG: Postcode callback for address page was wrong
- BUG: When reviews disabled, sort by still shows reviews in list
- BUG: undefined index: default on customer edit screen for multiple addresses


Added:
- Copied products are disabled by default and marked with [*]
- Google Base Feed support for currency (USD, EUR, GBP)
- Improved document class loading in header.php
- Enhanced currency library functions
- Alternating color rows for admin lists
- Added x_invoice_num to authorize.net
- Added support for discounts greater than product total for paypal standard
- Added Hawkey's improved version of Admin Order Edit. This version comes with the following improvements and limitations:
---- It will allow you to add or delete products and it will update the Subtotal, TAX and Total.
---- It will also recognize if a product is a Special even-tough it will not show the special price in the list
---- It will NOT recognize Quantity Discount.
---- It will NOT let you change the payment nor the shipment method.
---- It will NOT let you change the Shipping costs.
---- Be aware that if you change the sort order of the "Extension->Order Total" area, it will not map correctly to the values in previously edited orders.

Image


User avatar
Administrator

Posts

Joined
Tue Jul 22, 2008 3:02 am

Post by Johnathan » Sun Dec 12, 2010 11:38 pm

Qphoria wrote:OK... While waiting for 1.5.0, I'm going to release a 1.4.9.3 FULL and PATCH.
Sweet! That would be great, thanks. I just have two requests:
Qphoria wrote:- Copied products are disabled by default and marked with [*]
Is this at the end of the product name? If it's at the beginning the copied product won't necessarily show up on the same page as the product you just copied. (And you'll then have to go searching for it.)
Johnathan wrote:A very tiny bug that I assume affects all versions, though I've only confirmed it on 1.4.9.2:

If the customer's first or last name has a space in it, it will put the names on different lines in the invoice. For example, if the customer enters their first name as "John " and their last name as "Doe", the invoice will show:

John
Doe
123 Address St.
etc. etc.
I looked at the code for this error, and figure it must be somewhere in /admin/controller/sale/order.php around these lines:

Code: Select all

$shipping_address = str_replace(array("\r\n", "\r", "\n"), '<br />', preg_replace(array("/\s\s+/", "/\r\r+/", "/\n\n+/"), '<br />', trim(str_replace($find, $replace, $format))));
 

Code: Select all

$payment_address = str_replace(array("\r\n", "\r", "\n"), '<br />', preg_replace(array("/\s\s+/", "/\r\r+/", "/\n\n+/"), '<br />', trim(str_replace($find, $replace, $format))));
 
I can't quite figure out why, though. If you can see the solution and it's an easy fix, could you add it in as well?

Image Image Image Image Image


User avatar
Administrator

Posts

Joined
Fri Dec 18, 2009 3:08 am


Post by Qphoria » Mon Dec 13, 2010 2:19 am

Johnathan wrote: Is this at the end of the product name? If it's at the beginning the copied product won't necessarily show up on the same page as the product you just copied. (And you'll then have to go searching for it.)
What am I, slow? Of course it is at the end.
Johnathan wrote: I looked at the code for this error, and figure it must be somewhere in /admin/controller/sale/order.php around these lines:

Code: Select all

$shipping_address = str_replace(array("\r\n", "\r", "\n"), '<br />', preg_replace(array("/\s\s+/", "/\r\r+/", "/\n\n+/"), '<br />', trim(str_replace($find, $replace, $format))));
 

Code: Select all

$payment_address = str_replace(array("\r\n", "\r", "\n"), '<br />', preg_replace(array("/\s\s+/", "/\r\r+/", "/\n\n+/"), '<br />', trim(str_replace($find, $replace, $format))));
 
I can't quite figure out why, though. If you can see the solution and it's an easy fix, could you add it in as well?
Well if its a space thing I'd assume removing the "/\s\s+/", part from the regex would do it.. can you try that?

Image


User avatar
Administrator

Posts

Joined
Tue Jul 22, 2008 3:02 am

Post by Johnathan » Mon Dec 13, 2010 3:56 am

Qphoria wrote:What am I, slow? Of course it is at the end.
;D Just checking, in case you hadn't thought about it.
Qphoria wrote:Well if its a space thing I'd assume removing the "/\s\s+/", part from the regex would do it.. can you try that?
Removing the "/\s\s+/" caused some problems, but I think this fixes it:

Code: Select all

IN:
/admin/controller/sale/order.php

REPLACE:
'firstname' => $order_info['shipping_firstname'],
'lastname'  => $order_info['shipping_lastname'],

WITH:
'firstname' => trim($order_info['shipping_firstname']),
'lastname'  => trim($order_info['shipping_lastname']),

AND REPLACE:
'firstname' => $order_info['payment_firstname'],
'lastname'  => $order_info['payment_lastname'],

WITH:
'firstname' => trim($order_info['payment_firstname']),
'lastname'  => trim($order_info['payment_lastname']), 

Image Image Image Image Image


User avatar
Administrator

Posts

Joined
Fri Dec 18, 2009 3:08 am


Post by cmebd » Wed Dec 15, 2010 8:53 am

Qphoria wrote:OK... While waiting for 1.5.0, I'm going to release a 1.4.9.3 FULL and PATCH.
Any signs/ideas on when this might happen?

tks

A stupid question is the one you -don't- ask.........(Anon)

)C1.5.0.1 (IN devel)
OC V1.4.9.5
OC V1.4.9.2
OC V1.4.7
OC V1.3.4


User avatar
Active Member

Posts

Joined
Fri Nov 13, 2009 11:17 am
Location - Tasmania, Australia

Post by mystifier » Wed Dec 15, 2010 4:24 pm

The race is on. It looked as though v1.5 had it all the way but a late run by v1.4.9.3 has put it in strong contention. A few yards to the line and it is anyone's race.
Last edited by mystifier on Wed Dec 15, 2010 5:41 pm, edited 1 time in total.

Free v1.4.9 Extensions: Default Specials | Improved Search | Customer Activity Report | Customer Groups | Royal Mail With Handling | Improved Product Page | Random Products | Stock Report | All Products


User avatar
Active Member

Posts

Joined
Tue May 18, 2010 5:15 pm

Post by jty » Wed Dec 15, 2010 5:26 pm

Johnathan wrote:Removing the "/\s\s+/" caused some problems, but I think this fixes it:
I've just done a quick test on using rtrim on controller/checkout/guest_step_1.php ie get rid of the whitepsace at the point of saving the order, rather than at the point of printing the invoice.
It appears to be working. Guess I'll know soon enough if this doesn't work.

Thanks for finding this problem. It has baffled me for a couple of months but I didn't have the time to dig around and find the cause so I put up with being annoyed.

jty
Active Member

Posts

Joined
Sat Aug 30, 2008 8:19 am

Post by lucass » Fri Dec 17, 2010 5:19 pm

Hello

I am using Opencart 1.4.9.1 patched to 1.4.9.2.

In my store I have only one product at the moment. I added few options to this product (options were edited and saved many times), but suddenly system stopped working properly. When I go to product edit page, and click "Save" button (even without making any changes) I receive following error:

Code: Select all

2010-12-17 8:56:52 - PHP Notice:  Undefined index:  sort_order in /home/public_html/admin/model/catalog/product.php on line 136
2010-12-17 8:56:52 - PHP Warning:  Cannot modify header information - headers already sent by (output started at /home/public_html/admin/index.php:72) in /home/public_html/system/engine/controller.php on line 27
When I add new product, and click "Save", it works properly.

I am looking forward for Your response.

Best regards, Łukasz

Newbie

Posts

Joined
Fri Dec 17, 2010 5:05 pm

Post by Qphoria » Fri Dec 17, 2010 9:01 pm

Johnathan wrote: Removing the "/\s\s+/" caused some problems, but I think this fixes it:

Code: Select all

IN:
/admin/controller/sale/order.php

REPLACE:
'firstname' => $order_info['shipping_firstname'],
'lastname'  => $order_info['shipping_lastname'],

WITH:
'firstname' => trim($order_info['shipping_firstname']),
'lastname'  => trim($order_info['shipping_lastname']),

AND REPLACE:
'firstname' => $order_info['payment_firstname'],
'lastname'  => $order_info['payment_lastname'],

WITH:
'firstname' => trim($order_info['payment_firstname']),
'lastname'  => trim($order_info['payment_lastname']), 
Do you "think" or "know" ?

Image


User avatar
Administrator

Posts

Joined
Tue Jul 22, 2008 3:02 am

Post by Johnathan » Fri Dec 17, 2010 11:19 pm

I tested it in as many ways as I could and it worked every time as expected.

As jty wrote, it may be better to implement the trim() in the checkout process rather than right before displaying the invoice. That way the trimmed names get put in the database.

Image Image Image Image Image


User avatar
Administrator

Posts

Joined
Fri Dec 18, 2009 3:08 am


Post by Qphoria » Sat Dec 18, 2010 2:21 am

went with the trim. Thanks Both!

Image


User avatar
Administrator

Posts

Joined
Tue Jul 22, 2008 3:02 am

Post by Brook » Sun Dec 19, 2010 2:03 am

Just trying to understand.... in admin/controller/sale/order.php

I understand that you add the trim function to the lines above for first name and last name, why would you not also add the trim function to the rest of the elements in the shipping infor array and payment info array? (Ex. company, address_1, address_2, city, postcode etc...)

shipping info array

Code: Select all

				$replace = array(
					'firstname' => trim($order_info['shipping_firstname']),
					'lastname'  => trim($order_info['shipping_lastname']),
					'company'   => $order_info['shipping_company'],
					'address_1' => $order_info['shipping_address_1'],
					'address_2' => $order_info['shipping_address_2'],
					'city'      => $order_info['shipping_city'],
					'postcode'  => $order_info['shipping_postcode'],
					'zone'      => $order_info['shipping_zone'],
					'zone_code' => $order_info['shipping_zone_code'],
					'country'   => $order_info['shipping_country']
				);
payment info array

Code: Select all

				$replace = array(
					'firstname' => trim($order_info['payment_firstname']),
					'lastname'  => trim($order_info['payment_lastname']),					
					'company'   => $order_info['payment_company'],
					'address_1' => $order_info['payment_address_1'],
					'address_2' => $order_info['payment_address_2'],
					'city'      => $order_info['payment_city'],
					'postcode'  => $order_info['payment_postcode'],
					'zone'      => $order_info['payment_zone'],
					'zone_code' => $order_info['payment_zone_code'],
					'country'   => $order_info['payment_country']
				);

Active Member

Posts

Joined
Wed Feb 24, 2010 12:15 am
Who is online

Users browsing this forum: No registered users and 35 guests