Code to read fields from db for freightquote shipping module

[jimmie]

trying to add something like this in my shipping module

foreach ($products as $product) {
        $data = $db->Execute(
          "SELECT products_freightquote_length, products_freightquote_width, products_freightquote_height " .
          "FROM " . TABLE_PRODUCTS . " " .
          "WHERE products_id = " . (int)$product['id'] . " " .
          "LIMIT 1"
        );

To be read in this section

<Dimensions>
		<Length>ceil($data->fields['products_freightquote_length']),</Length>
		<Width>ceil($data->fields['products_freightquote_width']),</Width>
		<Height>ceil($data->fields['products_freightquote_height']),</Height>
		<Units>IN</Units>
	  </Dimensions>
    </RequestedPackageLineItems>
</RequestedShipment>
</RateRequest>";

what is the proper way

[jimmie]

try to add dimensions to my fedex module, this piece of code is from another module that uses the fields i want to use

      foreach ($products as $product) {
        $data = $db->Execute(
          "SELECT products_freightquote_length, products_freightquote_width, products_freightquote_height " .
          "FROM " . TABLE_PRODUCTS . " " .
          "WHERE products_id = " . (int)$product['id'] . " " .
          "LIMIT 1"
        );

Been try to add it in here with no luck

<?php

/**
 * Ceon Advanced Shipper Fedex Calculation class. 
 *
 * @package     ceon_advanced_shipper
 * @author      Waqas Hussain <waqas20######################>
 * @author      Conor Kerr <[email protected]>
 * @copyright   Copyright 2010-2012 Waqas Hussain
 * @copyright   Copyright 2007-2012 Ceon
 * @copyright   Portions Copyright 2003-2006 Zen Cart Development Team
 * @copyright   Portions Copyright 2003 osCommerce
 * @link        http://dev.ceon.net/web/zen-cart/advanced-shipper
 * @license     http://www.gnu.org/copyleft/gpl.html   GNU Public License V2.0
 * @version     $Id: class.AdvancedShipperFedExCalculator.php 981 2012-03-27 16:28:46Z conor $
 */

// {{{ AdvancedShipperFedExCalculator

/**
 * Connects to FedEx online calculator and gets quotes for the shipping methods enabled in the
 * configuration.
 *
 * @author      Waqas Hussain <waqas20######################>
 * @author      Conor Kerr <[email protected]>
 * @copyright   Copyright 2010-2012 Waqas Hussain
 * @copyright   Copyright 2007-2012 Ceon
 * @copyright   Portions Copyright 2003-2006 Zen Cart Development Team
 * @copyright   Portions Copyright 2003 osCommerce
 * @license     http://www.gnu.org/copyleft/gpl.html   GNU Public License V2.0
 */
class AdvancedShipperFedExCalculator
{
	// {{{ properties
	
	/**
	 * The configuration settings for this instance.
	 *
	 * @var     array
	 * @access  public
	 */
	var $_config = null;
	
	// }}}
	
	
	// {{{ Class Constructor
	
	/**
	 * Create a new instance of the AdvancedShipperFedExCalculator class
	 *
	 * @param   array     $fedex_config   An associative array with the configuration settings for
	 *                                    this instance.
	 */
	function AdvancedShipperFedExCalculator($fedex_config)
	{
		$this->_config = $fedex_config;
	}
	
	// }}}
	
	
	// {{{ quote()

	/**
	 * Contacts Fedex and gets a quote for the specified weight and configuration settings.
	 *
	 * @access  public
	 * @param   float     $weight    The weight of the package to be shipped.
	 * @param   float     $price     The total price of the items in package to be shipped.
	 * @param   array     $min_max   Any minimum/maximum limits which should be applied to the final
	 *                               rate calculated.
	 * @return  none
	 */
	function quote($weight, $price, $min_max)
	{
		global $order;
		
		if ($weight < 0.1) {
			$weight = 0.1;
		}
		
		$rate_info = $this->_getQuote($weight, $min_max);
		
		return $rate_info;
	}
	
	// }}}
	
	
	// {{{ _getQuote()
	
	/**
	 * Contacts FedEx, gets a quote, parses the response and builds the list of quotes.
	 *
	 * @access  protected
	 * @return  array|boolean   Array of results or boolean false if no results.
	 */
	function _getQuote($weight, $min_max)
	{
		global $db, $order, $currencies;
		
		if (strtolower($this->_config['server']) == 'p') {
			$fedex_url = 'https://gateway.fedex.com:443/xml';
		} else {
			$fedex_url = 'https://gatewaybeta.fedex.com:443/xml';
		}
		
		$fedex_key = $this->_config['fedex_key'];
		$fedex_password = $this->_config['fedex_password'];
		$fedex_account = $this->_config['fedex_account'];
		$fedex_meter = $this->_config['fedex_meter'];
		
		$shipper_country_sql = "
			SELECT
				countries_iso_code_2
            FROM
				" . TABLE_COUNTRIES . "
            WHERE
				countries_id = '" . (int) $this->_config['source_country'] . "';";
		
        $shipper_country_result = $db->Execute($shipper_country_sql);
		
		if (!$shipper_country_result->EOF) {
			$shipper_country = strtoupper($shipper_country_result->fields['countries_iso_code_2']);
		}
		
		$shipper_postcode = $this->_config['source_postcode'];
		$recipient_country = $order->delivery['country']['iso_code_2'];
		$recipient_postcode = $order->delivery['postcode'];
		$timestamp = gmdate('c');//'2010-08-06T21:48:17+06:00'
		$saturday_delivery = ($this->_config['shipping_saturday'] != 0);
		
		//BOF Residential Company Name Check
        $residential = ($order->delivery['company'] != '' ? 'false' : 'true');
        //EOF Residential Company Name Check
		
		$drop_off_type = strtoupper($this->_config['drop_off_type']);
		
		$packaging_type = strtoupper($this->_config['packaging_type']);
		
		$get_account_rates = ($this->_config['rate_request_types'] == 1 ||
			$this->_config['rate_request_types'] == 3 ? true: false);
		
		$get_list_rates = ($this->_config['rate_request_types'] == 2 ||
			$this->_config['rate_request_types'] == 3 ? true: false);
		
		$weight_units = strtoupper($this->_config['weight_units']);
		
		
		$request = "
<RateRequest xmlns='http://fedex.com/ws/rate/v16'>
	<WebAuthenticationDetail>
		<UserCredential>
			<Key>$fedex_key</Key>
			<Password>$fedex_password</Password>
		</UserCredential>
	</WebAuthenticationDetail>
	<ClientDetail>
		<AccountNumber>$fedex_account</AccountNumber>
		<MeterNumber>$fedex_meter</MeterNumber>
	</ClientDetail>
	<TransactionDetail>
		<CustomerTransactionId>ACTIVESHIPPING</CustomerTransactionId>
	</TransactionDetail>
	
	<Version>
		<ServiceId>crs</ServiceId>
		<Major>16</Major>
		<Intermediate>0</Intermediate>
		<Minor>0</Minor>
	</Version>
	
	<ReturnTransitAndCommit>true</ReturnTransitAndCommit>
	" . ($saturday_delivery ? '<VariableOptions>SATURDAY_DELIVERY</VariableOptions>' : '') . "
	
	<RequestedShipment>
		<ShipTimestamp>$timestamp</ShipTimestamp>
		<DropoffType>$drop_off_type</DropoffType>
		<PackagingType>$packaging_type</PackagingType>
		
		<Shipper>
			<Address>
				<PostalCode>$shipper_postcode</PostalCode>
				<CountryCode>$shipper_country</CountryCode>
			</Address>
		</Shipper>
		<Recipient>
			<Address>
			    <StreetLines>$recipient_street_address</StreetLines>
				<City>$recipient_city</City>
				<StateOrProvinceCode>$recipient_state</StateOrProvinceCode>
				<PostalCode>$recipient_postcode</PostalCode>
				<CountryCode>$recipient_country</CountryCode>	
			    <Residential>".$residential."</Residential>
			</Address>
		</Recipient>
		<ShippingChargesPayment>
<PaymentType>SENDER</PaymentType>
<Payor>
<AssociatedAccounts>
<AccountNumber></AccountNumber>
</AssociatedAccounts>
</Payor>
</ShippingChargesPayment>
		" . ($get_account_rates ? '<RateRequestTypes>NONE</RateRequestTypes>' : '') . "
		" . ($get_list_rates ? '<RateRequestTypes>LIST</RateRequestTypes>' : '') . "
		
		<CustomerSelectedActualRateType>PAYOR_LIST_PACKAGE</CustomerSelectedActualRateType>
		<PackageCount>1</PackageCount>
		<RequestedPackageLineItems>
		<SequenceNumber>1</SequenceNumber>
        <GroupPackageCount>1</GroupPackageCount>
			<Weight>
				<Units>$weight_units</Units>
				<Value>$weight</Value>
			</Weight>
			<Dimensions>
				<Length>ceil($data->fields['products_freightquote_length'])</Length>
				<Width>ceil($data->fields['products_freightquote_width'])</Width>
				<Height>ceil($data->fields['products_freightquote_height'])</Height>
				<Units>IN</Units>
			</Dimensions>
		</RequestedPackageLineItems>
	</RequestedShipment>
</RateRequest>";
		
		advshipper::debug("Data being sent to FedEx: <br />\n<br />\n" .
			nl2br(htmlentities($request)) . "<br />\n<br />\n", true);
		
		$body = $this->_post($fedex_url, $request);
		
		if (!$body) {
			advshipper::debug('Failed to connect to FedEx' . "\n\n");
			
			return array('error' => 'Failed to connect to FedEx');
		}
		
		// Strip namespace prefixes
		$body =
			preg_replace(array('/<[a-zA-Z0-9]+:/', '/<\/[a-zA-Z0-9]+:/'), array('<', '</'), $body); 
		
		advshipper::debug("Data received from FedEx (with namespaces removed): <br />\n<br />\n" .
			nl2br(htmlentities($body)) . "<br />\n<br />\n", true);
		
		$xml = @simplexml_load_string($body);
		
		if (!$xml) {
			advshipper::debug('Unable to parse XML returned from FedEx' . "<br />\n<br />\n");
			
			return array(
				'error' => 'Invalid response from FedEx: '. $body
				);
		}
		
		if (strtolower($xml->getName()) == 'fault') {
			$error_code = $xml->detail->fault->errorCode;
			$reason = $xml->detail->fault->reason;
			
			return array(
				'error' => 'FedEx: ' . $error_code . ': ' . $reason
				);
		}
		
		$severity = $xml->HighestSeverity;
		$message = $xml->Notifications->Message;
		
		// Check for problem with postcode
		if ($severity == 'ERROR' &&
				strpos($message, 'Destination postal code missing or invalid') !== false) {
			return array(
				'error' => MODULE_ADVANCED_SHIPPER_ERROR_INVALID_POSTCODE
				);
		}
		
		if ($severity != 'SUCCESS' && $severity != 'NOTE' && $severity != 'WARNING') {
			advshipper::debug('Error occured when attempting to get response from FedEx. Severity' .
				' was ' . $severity . "<br />\n<br />\n");
			
			return array(
				'error' => "FedEx: $message"
				);
		}
		
		$rate_info = array();
		
		foreach ($xml->RateReplyDetails as $child) {
			$service_type = trim(strtolower((string) $child->ServiceType));
			
			$service_name = constant('MODULE_ADVANCED_SHIPPER_TEXT_FEDEX_' .
				str_replace('FEDEX_', '', strtoupper($service_type)));
			
			if ($this->_config['shipping_service_' . $service_type] == 0) {
				advshipper::debug('FedEx ' . $service_name . ' service not to be offered as it is' .
					" not enabled in the FedEx configuration for the region.<br />\n<br />\n",
					true);
				
				continue;
			}
			
			if ($child->AppliedOptions == 'SATURDAY_DELIVERY') {
				$service_name .= MODULE_ADVANCED_SHIPPER_TEXT_FEDEX_SATURDAY_DELIVERY;
			}
			
			$rate = (string) $child->RatedShipmentDetails->ShipmentRateDetail->TotalNetCharge->
				Amount;
			
			//If the method is configured to use list rates find the payor_list_package rate otherwise use account rates - RB
			if ($get_list_rates == '1') {
				foreach ($child->RatedShipmentDetails as $ratedetail) {
					$ratetype =	$ratedetail->ShipmentRateDetail->RateType;
					if ($ratetype == 'PAYOR_LIST_PACKAGE') {
					$rate = (string) $ratedetail->ShipmentRateDetail->TotalNetCharge->Amount;
					}
				}
			}
			//End of account rate change
			
			// Rate is always returned in currency of account holder. Must convert to currency in
			// use
			$rate_currency = strtoupper((string) $child->RatedShipmentDetails->ShipmentRateDetail->
				TotalNetCharge->Currency);
			
			// Get the current currency in which the customer is viewing prices
			$current_display_currency = strtoupper($_SESSION['currency']);
			
			if ($rate_currency != $current_display_currency) {
				// Must convert from FedEx currency to base currency
				$rate_currency_conv_rate = $currencies->get_value($rate_currency);
				
				if (is_null($rate_currency_conv_rate) || !is_numeric($rate_currency_conv_rate)) {
					// FedEx currency doesn't have a conversion value, can't proceed!
					return array(
						'error' => sprintf(
							MODULE_ADVANCED_SHIPPER_ERROR_FEDEX_CURRENCY_CONV_VALUE_MISSING,
							$rate_currency)
						);
				} else if ($rate_currency_conv_rate <= 0) {
					return array(
						'error' => sprintf(
							MODULE_ADVANCED_SHIPPER_ERROR_FEDEX_CURRENCY_CONV_VALUE_INVALID,
							$rate_currency)
						);
				}
				
				$rate_base_conv_rate = 1 / $currencies->get_value($rate_currency);
				
				$rate = number_format($rate * $rate_base_conv_rate, 2, '.', '');
			}
			
			if ($min_max != false) {
				// Apply the limit(s) to the rate
				$rate_limited = advshipper::calcMinMaxValue($rate, $min_max['min'],
					$min_max['max']);
				
				if ($rate_limited != $rate) {
					$rate = $rate_limited;
				}
			}
			
			$rate_info[] = array(
				'rate' => $rate,
				'rate_components_info' => array(
					array(
						'value_band_total' => $rate,
						'individual_value' => null,
						'num_individual_values' => $weight,
						'additional_charge' => null,
						'calc_method' => ADVSHIPPER_CALC_METHOD_FEDEX
						)
					),
				'rate_extra_title' => MODULE_ADVANCED_SHIPPER_TEXT_FEDEX_TITLE_PREFIX .
					$service_name
				);
		}
		
		if (count($rate_info) == 0) {
			// No quotes
			return false;
		}
		
		return $rate_info;
	}
	
	// }}}
	
	
	// {{{ _post()
	
	/**
	 * Do an HTTP POST call on a URL with given data.
	 *
	 * @access  protected
	 * @param   string    $url   The URL to send the request to.
	 * @param   string    $data  The data to send in the request.
	 * @return  string    Data or false on error
	 */
	function _post($url, $data)
	{
		$ch = curl_init($url) or die('Failed to load curl');
		
		curl_setopt($ch, CURLOPT_RETURNTRANSFER, TRUE);
		//curl_set_opt($ch, CURLOPT_HEADER, FALSE);
		curl_setopt($ch, CURLOPT_POST, TRUE);
		curl_setopt($ch, CURLOPT_POSTFIELDS, $data);
		curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, FALSE);
		
		$data = curl_exec($ch);
		curl_close($ch);
		
		return $data;
	}
	
	// }}}
}

// }}}

?>

Updated Post to show complete code

[mc12345678]

Thing with this is and as said it is code from something else that is attempted to be added, the issue is basically variable scope/definition. The first piece of code (that I didn't see in the second) assumes that the products ($products) are known and in some sort of data structure such as returned by a sql query. That code then takes information about each product from the database and retrieves it as $data. This current module doesn't look like it evaluates the individual products, but instead the weight of the shipment, which I believe is calculated elsewhere before getting to this module/section of code based on the $weight being passed into the function(s).

[jimmie]

When i pasted the code in i reverted it back to the original code.( taking out some of my changes, they didnt work)
when i put numbers here it changes my rate quote, so i need a way to read those db entries and and fill in the dimensions,

			<Dimensions>
				<Length>12</Length>
				<Width>12</Width>
				<Height>12</Height>
				<Units>IN</Units>
			</Dimensions>

[Ajeh]

And if you change this section of code to something like:

			<Dimensions>
				<Length>" . ceil($data->fields['products_freightquote_length']) . "</Length>
				<Width>" . ceil($data->fields['products_freightquote_width']) . "</Width>
				<Height>" . ceil($data->fields['products_freightquote_height']) . "</Height>
				<Units>IN</Units>
			</Dimensions>

[jimmie]

ajeh this is what was returned

<Dimensions>
<Length>0</Length>
<Width>0</Width>
<Height>0</Height>
<Units>IN</Units>
</Dimensions>

[Ajeh]

This needs something in that section for the $data as that is not defined in your code to get the data that should be there to use the change that I posted ...

I do not know what you need to create that, I just posted what you needed in the code to use set that information if the $data existed ...

[lhungil]

What are the current database fields for the database table "products" (TABLE_PRODUCTS)?
Is data present in the "additional" fields for the product being tested? What is the data?

When multiple products are in an order how will you handle them? Do they all go into one shipping package? Each product in their own shipping package? Different packaging depending on the quantity of each product? Using some sort of algorithm to place multiple products into a package (resulting in less packages than products)?



NOTE: I would argue strongly against adding "shipping" dimensions to a product. Why (data domain reason)? Because shipping dimensions are not dependent on the product alone and based on the packaging used when shipping. And the packaging used will be based on: product dimensions, selected packaging, and selected shipping carrier. Another reason (KISS reason)? When one alters an existing database table, upgrades and troubleshooting now need to take additional steps based upon the alterations. This can increase the amount of work required to maintain and upgrade Zen Cart in the future.

[jimmie]

products_freightquote_length, products_freightquote_width, products_freightquote_height,
These products can only ship by themselves no other packaging can go with them.
i havent setup all product dimensions yet but this one is
http://floorz-n-more.com/index.php?m...oducts_id=1843
Length 44
Width 16
height 3

[lhungil]

... These products can only ship by themselves no other packaging can go with them. ...

Are you saying...

Modifications have been made to Zen Cart:

• Customer is not allowed to add this product to their shopping cart when a product is already in the shopping cart.
• Customer is not allowed to add other products to their shopping cart when this product is already in the cart.
• This product is restricted to only allow a quantity of one.

OR

Multiple (mixed) products are allowed and:

• Customer can have multiple products in the shopping cart.
• Customer can add quantities greater than one.
• One package (with dimensions) per quantity of each product for shipping.