Problem/Motivation

Right now, the OrderSubscriber uses the current request timestamp as the start time of the created subscription. However, if an administrator is creating an order through the OMS with an alternative placed date, that date should be used, not the current request date.

Proposed resolution

Check to see if the order already has a placed timestamp and use that instead.

Comments

rszrama created an issue. See original summary.

  • rszrama committed bc057766 on 8.x-1.x 
    Issue #3539844 by rszrama: Use a previously existing placed date on an...
rszrama’s picture

rszrama commented
Status: Active » Fixed

Committed, patch autogenerated by Cursor after just copying the relevant if statement from the order module and entering a new line in the subscriber. It detected from my clipboard what I intended to do and wrote the comment and then adjusted the setter. Magic!

jsacksick’s picture

jsacksick commented

FYI, we also have a built-in method for that. See Order::getCalculationDate().

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

jsacksick’s picture

jsacksick commented

@rszrama: Just FYI, the tests have started failing since this change got committed.