[
https://issues.apache.org/jira/browse/OFBIZ-270?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12466952 ]
Jacques Le Roux commented on OFBIZ-270:
---------------------------------------
Please Daniel,
First of all read
http://docs.ofbiz.org/display/OFBADMIN/OFBiz+Contributors+Best+Practices. You will see (not sure all is explained in this document although, hence this quick complement) that Apache policy does not allow commiters to apply patches if the author has not checked the "ASF granted licence". You will see when *attaching* your patch. Moreover your patch is wrapped in middle of lines, not easy to catch with.
Mmm, I see the problem now. This issue is closed, I reopen it to allow you to attach the file, sorry for the sermon ;o)
Thanks
> USPS Rate Estimation
> --------------------
>
> Key: OFBIZ-270
> URL:
https://issues.apache.org/jira/browse/OFBIZ-270> Project: Apache OFBiz (The Open for Business Project)
> Issue Type: Bug
> Components: product
> Affects Versions: SVN trunk
> Environment: Any
> Reporter: Marco Risaliti
> Assigned To: Jacques Le Roux
> Priority: Minor
> Fix For: SVN trunk
>
> Attachments: shipping.patch
>
>
> This is a copy of the old Jira issue
http://jira.undersunconsulting.com/browse/OFBIZ-657 from Daniel Kunkel
> ===========================================================
> The USPS automatic Rate Inquiry module has numerous flaws.
> I was able to find and fix them, albeit, with potentially unacceptable techniques.
> Specifically:
> 1.) USPS now uses Get and does not Post any longer.
> 2.) Pounds must be a integer number.
> 3.) The calculation for breaking a weight into pounds and ounces was inaccurate. (UPS module too)
> 4.) The HttpClient HTTP GET was inserting & between the value pairs instead of &
> !!!! My fix for this issue may not be appropriate!
> 5.) No provision for converting other units of measure to Pounds. Other parts of the program seem to default to the ounce, while shipping defaulted to the pound.
> Remaining Issues...
> Must find a way to disable the estimator during an error. The system currently defaults to $1, which is a believeable, yet money loosing postage.
> Need to add International Shipping Support.
> Finally... This is my first patch contribution... It will need extra time and attention to verify!
> Thanks
> Daniel
>
>
> All Comments Work Log Change History Sort Order:
> Comment by Daniel Kunkel [08/Jan/06 02:13 AM] [ Permlink ]
> My fixes which SEEM to work. Your milage may vary.
> Comment by Marco Risaliti [13/Sep/06 01:43 AM] [ Permlink ]
> Can be closed ?
> Thanks
> Marco
> Comment by Daniel Kunkel [13/Sep/06 02:09 AM] [ Permlink ]
> I think it would be a waste to close this issue without first applying the patch... Granted it was my first patch and even I don't like the code, but it did get the domestic USPS postal calculator working which has been a real boon for our business. Furthermore it allowed for the conversion of product weights to lbs for the usps, another feature we needed.
> PS. I do have an iCLA signed.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
Locked
