Big Royal Mail 3.0

[brittainmark]

install3.2 sql file seems to contain a syntax error and therefore cannot be installed until it's corrected?

I have check the SQL on my system and it is fine. Just incase here is the script.

--
-- Set up the Attributes for Big royal Mail delivery options.
--
-- Create or find the Delivery option
--
SELECT @options_id:=ifnull(max(products_options_id)+1,1) FROM products_options;

SELECT @options_id := ifnull(products_options_id, @options_id) FROM products_options WHERE products_options_name="Minimum Delivery Method:" ;

REPLACE INTO products_options (products_options_id, language_id, products_options_name, products_options_sort_order, products_options_type, products_options_length, products_options_comment, products_options_size, products_options_images_per_row, products_options_images_style, products_options_rows) VALUES
(@options_id, 1, 'Minimum Delivery Method:', 0, 5, 32, '', 32, 0, 0, 0);

SELECT @options_id := products_options_id FROM products_options WHERE products_options_name="Minimum Delivery Method:" ;

--
-- Set up the option values
--
-- Large Letter
--
SELECT @LargeLetter_id := ifnull(max(products_options_values_id)+1,1) FROM products_options_values;

SELECT @LargeLetter_id := ifnull(products_options_values_id, @LargeLetter_id) FROM products_options_values WHERE products_options_values_name="Large Letter" ;

REPLACE INTO products_options_values (products_options_values_id, language_id, products_options_values_name, products_options_values_sort_order) VALUES
(@LargeLetter_id, 1, 'Large Letter', 10);

SELECT @LargeLetter_id := products_options_values_id FROM products_options_values WHERE products_options_values_name="Large Letter" ;

--
-- Small Parcel
--
SELECT @SmallParcel_id := ifnull(max(products_options_values_id)+1,1) FROM products_options_values;

SELECT @SmallParcel_id := ifnull(products_options_values_id, @SmallParcel_id) FROM products_options_values WHERE products_options_values_name="Small Parcel" ;

REPLACE INTO products_options_values (products_options_values_id, language_id, products_options_values_name, products_options_values_sort_order) VALUES
(@SmallParcel_id, 1, 'Small Parcel', 20);


SELECT @SmallParcel_id := products_options_values_id FROM products_options_values WHERE products_options_values_name="Small Parcel" ;

--
-- Medium Parcel
--
SELECT @MediumParcel_id := ifnull(max(products_options_values_id)+1,1) FROM products_options_values;

SELECT @MediumParcel_id := ifnull(products_options_values_id, @MediumParcel_id) FROM products_options_values WHERE products_options_values_name="Medium Parcel" ;

REPLACE INTO products_options_values (products_options_values_id, language_id, products_options_values_name, products_options_values_sort_order) VALUES
(@MediumParcel_id, 1, 'Medium Parcel', 30);


SELECT @MediumParcel_id := products_options_values_id FROM products_options_values WHERE products_options_values_name="Medium Parcel" ;

--
-- Parcel Force
--
SELECT @ParcelForce_id := ifnull(max(products_options_values_id)+1,1) FROM products_options_values;

SELECT @ParcelForce_id := ifnull(products_options_values_id, @ParcelForce_id) FROM products_options_values WHERE products_options_values_name="Special Delivary or Parcel Force" ;

REPLACE INTO products_options_values (products_options_values_id, language_id, products_options_values_name, products_options_values_sort_order) VALUES
(@ParcelForce_id, 1, 'Special Delivary or Parcel Force', 40);


SELECT @ParcelForce_id := products_options_values_id FROM products_options_values WHERE products_options_values_name="Special Delivary or Parcel Force" ;

--
-- Link the option and option values.
--
SELECT @Pov2po_id := ifnull(max(products_options_values_to_products_options_id)+1,1) FROM products_options_values_to_products_options;

SELECT @Pov2po_id := ifnull(products_options_values_to_products_options_id, @Pov2po_id) FROM products_options_values_to_products_options 
WHERE products_options_id= @options_id AND products_options_values_id = @LargeLetter_id;
REPLACE INTO products_options_values_to_products_options (products_options_values_to_products_options_id, products_options_id, products_options_values_id) VALUES
(@Pov2po_id, @options_id, @LargeLetter_id);

SELECT @Pov2po_id := ifnull(max(products_options_values_to_products_options_id)+1,1) FROM products_options_values_to_products_options;

SELECT @Pov2po_id := ifnull(products_options_values_to_products_options_id, @Pov2po_id) FROM products_options_values_to_products_options 
WHERE products_options_id= @options_id AND products_options_values_id = @SmallParcel_id;
REPLACE INTO products_options_values_to_products_options (products_options_values_to_products_options_id, products_options_id, products_options_values_id) VALUES
(@Pov2po_id, @options_id, @SmallParcel_id);

SELECT @Pov2po_id := ifnull(max(products_options_values_to_products_options_id)+1,1) FROM products_options_values_to_products_options;

SELECT @Pov2po_id := ifnull(products_options_values_to_products_options_id, @Pov2po_id) FROM products_options_values_to_products_options 
WHERE products_options_id= @options_id AND products_options_values_id = @MediumParcel_id;
REPLACE INTO products_options_values_to_products_options (products_options_values_to_products_options_id, products_options_id, products_options_values_id) VALUES
(@Pov2po_id, @options_id, @MediumParcel_id);

SELECT @Pov2po_id := ifnull(max(products_options_values_to_products_options_id)+1,1) FROM products_options_values_to_products_options;

SELECT @Pov2po_id := ifnull(products_options_values_to_products_options_id, @Pov2po_id) FROM products_options_values_to_products_options 
WHERE products_options_id= @options_id AND products_options_values_id = @ParcelForce_id;
REPLACE INTO products_options_values_to_products_options (products_options_values_to_products_options_id, products_options_id, products_options_values_id) VALUES
(@Pov2po_id, @options_id, @ParcelForce_id);

-- 
-- Add the insurance rates for Special delivery if installed
--
INSERT ignore INTO configuration (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, set_function, date_added)
SELECT 'Insurance rates ', 'MODULE_SHIPPING_RMSPECIALDELIVERY_ZONES_INSURE', '500:0,1000:1,2500:3', 'example: 200:1.2 means values less than or equal to £200 would cost £1.20. to insure. 100+:4.5 means that each additional £100 costs £4.50 to insure.', '6', '0', 'zen_cfg_textarea(', now()
FROM configuration WHERE configuration_key  = "MODULE_SHIPPING_RMSPECIALDELIVERY_STATUS";
  
INSERT ignore INTO configuration (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, set_function, date_added)
SELECT 'Insurance rates ', 'MODULE_SHIPPING_RMSPECIALDELIVERY9AM_ZONES_INSURE', '50:0,2.2:1000,5.7:2500', 'example: 200:1.2 means values less than or equal to £200 would cost £1.20. to insure. 100+:4.5 means that each additional £100 costs £4.50 to insure.', '6', '0', 'zen_cfg_textarea(', now()
FROM configuration WHERE configuration_key  = "MODULE_SHIPPING_RMSPECIALDELIVERY9AM_STATUS";

-- 
-- Update max order value for special to 2500
-- 
UPDATE configuration SET configuration_value = 2500 WHERE configuration_key  = "MODULE_SHIPPING_RMSPECIALDELIVERY_MAX_ORDERVALUE";

UPDATE configuration SET configuration_value = 2500 WHERE configuration_key  = "MODULE_SHIPPING_RMSPECIALDELIVERY9AM_MAX_ORDERVALUE";

[brittainmark]

I note that the instructions say to upload BigRoyalMail_V3.2.1/images/* but there are no images in the zip file.

Whoops. I was a bit rushed. I will rerelease the Images. They have not changed.

[brittainmark]

Just speculating here but the error mentions a product option called 'minimum deliv'

The README.htm for a fresh installation talks about setting the minimum delivery method via Catalog>Categories/Products> but having followed the UPGRADE.htm instructions I don't see anything about minimum delivery on the product options.

I've searched via developers toolkit for some of the code fragments in the error above and they only occur in /includes/modules/shipping/BigRoyalMail/quote.php so I tend to think the error is in this file.

Please see http://www.zen-cart.com/showthread.p...64#post1227764 to see about the new method I introduced at version 3.1. I will look at putting in some code to produce a meaningful error.

[brittainmark]

Found a bug connected with attributes.

Change the quote.php module in

YOUR_SHOPPING_CART_DIRECTORY/includes/modules/shipping/BigRoyalMail/quote.php

As a temporary fix replace

$attribute_postage_select = 'SELECT ifnull(max(pov.products_options_values_sort_order), 0) as postage_attribute FROM '.TABLE_PRODUCTS_OPTIONS_VALUES.' pov INNER JOIN '.
			TABLE_PRODUCTS_ATTRIBUTES.' pa ON pov.products_options_values_id = pa.options_values_id INNER JOIN '.
			TABLE_PRODUCTS_OPTIONS.' po ON po.products_options_id = pa.options_id WHERE pa.products_id IN ('.$products_list.
			') AND po.products_options_name = "Minimum Delivery Method:"';

with

$attribute_postage_select = 'SELECT 0';

This should work with 1.5 versions of Zen Cart I am not sure if it works with 1.3 versions but suspect it will.

[brittainmark]

I have found the issue. it was connected with attributes.
You need to change the quote.php which can be found in YOUR_SHOPPING_CART_DIRECTORY/includes/modules/shipping/BigRoyalMail
you need to replace line 57

$products_list = $_SESSION['cart']->get_product_id_list();

with

	$products_list = '';
	if (is_array($_SESSION['cart']->contents)) {
		reset($_SESSION['cart']->contents);
		while (list($products_id, ) = each($_SESSION['cart']->contents)) {
			$products_list .= ', ' . zen_get_prid(zen_db_input($products_id));
		}
	}
	$products_list = substr($products_list, 2);

[Some Bloke]

Hi Mark,

I just installed your latest update v3.2.1 in a test shop. (It seems to work fine in Zen Cart v1.3.9 by the way.)

Unfortunately, there seems to be one service missing in my installation, "International Signed" (which used to be International "signed for").

Royal Mail have really complicated things with this new shuffle, they seem to have made three overlapping services out of one!
Now we have "International Signed", "International Tracked & Signed" and "International Tracked". Strangely, all at the same price unless you send to Europe using the latter one (with no signature provided), when they add extra 20% vat.

Thanks for the contribution. You must have incredible patience.

[Some Bloke]

Sorry, my mistake, I just found it. There was a line with no description, just "ver. 3.2.1" in red letters that I'd overlooked. Clicked on it and that was the one I'd been looking for.

[brittainmark]

That suggest that you may be missing a language file or the file is the corrupt. The file should be in YOUR_SHOPPING_CART_DIRECTORY/includes/languages/english/modules/shipping/ and is called rmamparcelsf.php.

And Yes you are right it is crazy. I also only had a weeks notice of the change because parcelforce did not release their rates until the Monday before the change. You are right with 3 methods with the same price. The International Tracked and Signed is supposed to be faster than the other two signed and tracked.

If you use attributes at all please ensure that you apply the fix above http://www.zen-cart.com/showthread.p...02#post1243102. I am just setting up a 1.3.8a to try to test it. Glad to here that it works on 1.3.9.
Finally if you have updated from 3.0.1 you will need to apply RemoveSpecial.sql to remove the extra special delivery method from the configuration files.

[Some Bloke]

Hi Mark,

I checked that file and it's not really a problem, just that the text hasn't been fully filled in - obviously a "holding" dot. First line reads:


define('MODULE_SHIPPING_RMAMPARCELSF_TEXT_TITLE', '.');


Looks like the same for rmspecialdelivery file also.

[brittainmark]

Hi Mark,

I checked that file and it's not really a problem, just that the text hasn't been fully filled in - obviously a "holding" dot. First line reads:


define('MODULE_SHIPPING_RMAMPARCELSF_TEXT_TITLE', '.');


Looks like the same for rmspecialdelivery file also.

Air mail signed for should be

define('MODULE_SHIPPING_RMAMPARCELSF_TEXT_TITLE', 'Royal Mail <span style="color:red; font-weight:bold">International Signed </span><span style="font-weight: normal; font-style: italic">&quot;Small Parcel&quot;</span>');

Specialdelivery

define('MODULE_SHIPPING_RMSPECIALDELIVERY_TEXT_TITLE', 'Royal Mail <b>special delivery&reg;</b> <span style="font-weight:normal">next day</span>');

Both appear OK in the package. Are you using the override system? If so that could be the reason for them being blank.