View Issue Details

IDProjectCategoryView StatusLast Update
0003422OXID eShop (all versions)2. ----- eShop backend (admin) -----public2012-01-28 14:55
ReporterAndreHerrmann Assigned To 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version4.5.5 revision 40299 
Fixed in Version4.5.7 revision 41909 
Summary0003422: ordersentdate will never be shown
DescriptionHello,

yesterday I updated from PE 4.5.4 to pe 4.5.5. It seems that we have a feature-bug ;-)

The new method canResetShippingDate() in order_overview checks the sentdate on beeing bigger than 1 which will lead to be never true and following the admin user always gets an: "Not sent yet" message.

Additional InformationCurrent:
    /**
     * Get information about shipping status
     *
     * @return bool
     */
    public function canResetShippingDate()
    {
        $oOrder = oxNew( "oxorder" );
        $blCan = false;
        if ( $oOrder->load( $this->getEditObjectId() ) ) {
            if ( $oOrder->oxorder__oxstorno->value == "0" && $oOrder->oxorder__oxsenddate->value>"1" ) {
                $blCan = true;
            }
        }
        return $blCan;
    }


Patch which worked for me:
    /**
     * Get information about shipping status
     *
     * @return bool
     */
    public function canResetShippingDate()
    {
        $oOrder = oxNew( "oxorder" );
        $blCan = false;
        if ( $oOrder->load( $this->getEditObjectId() ) ) {
            if ( $oOrder->oxorder__oxstorno->value == "0" && $oOrder->oxorder__oxsenddate->value>"0000-00-00 00:00:00" ) {
                $blCan = true;
            }
        }
        return $blCan;
    }
TagsNo tags attached.
ThemeBoth
BrowserAll
PHP Versionany
Database Versionany

Activities

dainius.bigelis

2011-12-28 15:57

reporter   ~0005543

@Developers:
we checked from functionality side: the buttons "Ship now" and "Reset Shipping Date" works (one gives the shipping date to order, other resets it shows "Order not shipped yet"). So we don't recognize the issue in this case.
But please check from the source code if it's needed to fix the "if" statement to check the date "0000-00-00 00:00:00" instead of "1".