Edit Orders v4.0 Support Thread

[lat9]

Just wondering if there are any plans to update this module to be compatible with the new admin sanitizer, as per this thread:

https://www.zen-cart.com/showthread.php?219747

Thanks!

When the solution is posted to that thread, my guess is that no further change will be required of Edit Orders.

[DrByte]

Please test it: https://www.zen-cart.com/showthread....77#post1307377

[lat9]

Will do; it'll be tomorrow.

[swguy]

Has anyone tried adding multiline comments during an order edit? It seems to keep the ASCII carriage control in the database. So instead of

Line1
Line2
Line3

You get

Line1\r\nLine2\r\nLine3

[lat9]

Will do; it'll be tomorrow.

No go. I've merged the EO 4.1.4 code with the 3/29 update for Zen Cart 1.5.5, added a product to the order and saw that the foreach log was generated. Applied wilt's update and added another product to the order -- still getting that warning.

I added a product with attributes (Matrox G200 MMS) and the attributes don't show upon completion of the addition. I made a teeny change to the requestSanitizer class to also dump the $_GET and $_POST variables on entry; here's the log that's created after I've chosen the attributes to add to the product:

Mar-30-2016 09:42:02
=================================

Running Admin Sanitizers: Array
(
    [page] => 1
    [oID] => 3
    [action] => add_prdct
)
Array
(
    [securityToken] => 929ab5ecab8c7436fbd535d9fc431c06
    [id] => Array
        (
            [3] => Array
                (
                    [value] => 4
                    [type] => 0
                )

            [4] => Array
                (
                    [value] => 3
                    [type] => 0
                )

        )

    [add_product_categories_id] => 4
    [add_product_products_id] => 1
    [search] => 
    [step] => 4
)


SANITIZER BUILTIN == filterSimpleAlphanumPlus

PROCESSING SIMPLE_ALPHANUM_PLUS == action

PROCESSING SIMPLE_ALPHANUM_PLUS == oID

PROCESSING SIMPLE_ALPHANUM_PLUS == page

SANITIZER BUILTIN == filterConvertInt

PROCESSING CONVERT_INT (POST) == id

SANITIZER BUILTIN == filterFileDirRegex

SANITIZER BUILTIN == filterAlphanumDashUnderscore

PROCESSING ALPHANUM_DASH_UNDERSCORE (GET) == action

PROCESSING ALPHANUM_DASH_UNDERSCORE (GET) == page

SANITIZER BUILTIN == filterWordsAndSymbolsRegex

SANITIZER BUILTIN == filterMetaTags

SANITIZER BUILTIN == filterSanitizeEmail

SANITIZER BUILTIN == filterProductDescRegex

PROCESSING PRODUCT_DESC_REGEX == products_description

PROCESSING PRODUCT_DESC_REGEX == coupon_desc

PROCESSING PRODUCT_DESC_REGEX == file_contents

PROCESSING PRODUCT_DESC_REGEX == categories_description

PROCESSING PRODUCT_DESC_REGEX == message_html

PROCESSING PRODUCT_DESC_REGEX == banners_html_text

PROCESSING PRODUCT_DESC_REGEX == pages_html_text

PROCESSING PRODUCT_DESC_REGEX == comments

SANITIZER BUILTIN == filterProductUrlRegex

SANITIZER BUILTIN == filterCurrencyValueRegex

SANITIZER BUILTIN == filterProductNameDeepRegex

SANITIZER BUILTIN == filterStrictSanitizeValues

PROCESSING STRICT_SANITIZE_VALUES == securityToken

PROCESSING STRICT_SANITIZE_VALUES == id

PROCESSING STRICT_SANITIZE_VALUES == add_product_categories_id

PROCESSING STRICT_SANITIZE_VALUES == add_product_products_id

PROCESSING STRICT_SANITIZE_VALUES == search

PROCESSING STRICT_SANITIZE_VALUES == step

PROCESSING STRICT_SANITIZE_VALUES == page

PROCESSING STRICT_SANITIZE_VALUES == oID

PROCESSING STRICT_SANITIZE_VALUES == action

SANITIZER BUILTIN == filterStrictSanitizeKeys

[lat9]

Has anyone tried adding multiline comments during an order edit? It seems to keep the ASCII carriage control in the database. So instead of

Line1
Line2
Line3

You get

Line1\r\nLine2\r\nLine3

Yep, we've got some "over preparation" of the data; I'm getting " output when the input order contained simply "

[lat9]

OK, I've created a GitHub repository (https://github.com/lat9/edit_orders) for the plugin, initialized with EO 4.1.4 and updated with each of the bugfixes that I found while painfully scrolling through the postings.

I'm not sending this to the Zen Cart Plugins until I've had a chance to fully vet the changes; any help would be appreciated. Please continue to use this support-thread to report any issues found, making sure that you identify that you're using the GitHub version and note the date/time that you downloaded the repo's file-set.

[lat9]

I've updated the EO github site with a work-around for the Zen Cart 1.5.5 admin-sanitizer issue and verified that it's operating properly on a fresh Zen Cart 1.5.5-20160329 install with EO only. Additional eyes would help. I'll package this up as EO 4.1.4a sometime next week unless some calamity hits.

[margecc]

I was wondering if anyone would have a chance to look at my post #869 of page 87 (https://www.zen-cart.com/showthread....17#post1306017). I am having problems with my taxes being messed up and I can not figure out what file I've screwed up.

Any help would be greatly appreciated.

[lat9]

Colleen, check your PMs.