Differential D51

MINC-119 - Reset more fields during cloning
ClosedPublic
Actions

Authored by alex on Mar 20 2015, 10:01 AM.

Details

Reviewers

glebs

Commits

rMINC16244: Fixes MINC-119 - Reset more fields during cloning

Test Plan

create an order and process it
clone that order
confirm that fields, mentioned in the changed code are reset to their default values for the cloned order

Diff Detail

Repository

rINP In-Portal

Branch

/in-portal/branches/5.2.x

Lint

Lint OK

Unit

No Unit Test Coverage

Event Timeline

alex updated this revision to Diff 126.Mar 20 2015, 10:01 AM

alex retitled this revision from to MINC-119 - Reset more fields during cloning.

alex updated this object.

alex edited the test plan for this revision. (Show Details)

alex added 1 JIRA issue(s): MINC-119.

Herald added a reviewer: glebs. · View Herald TranscriptMar 20 2015, 10:01 AM

Removed assignment by reference (=&), that was introduced in previous patch.

glebs accepted this revision.Apr 27 2015, 2:29 AM

glebs edited edge metadata.

This revision is now accepted and ready to land.Apr 27 2015, 2:29 AM

Closed by commit rMINC16244: Fixes MINC-119 - Reset more fields during cloning (authored by alex). · Explain WhySep 2 2015, 3:31 PM

This revision was automatically updated to reflect the committed changes.

alex edited edge metadata.Mar 10 2016, 6:15 AM

alex added a project: Restricted Project.

Revision Contents
Changeset List

			Path	Packages
M			modules/in-commerce/units/orders/orders_event_handler.php (20 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	126	16128		Mar 20 2015, 10:01 AM	★	★
Diff 2	127	16128	Removed assignment by reference (`=&`), that was introduced in previous patch.	Mar 20 2015, 10:03 AM	★	★
Diff 3	290	16242	Commit rMINC16244	Sep 2 2015, 3:31 PM	★	★

Commit	Parents	Author	Summary	Date
#fake-hash			Ignore it :) (Show More…)

Diff 127

View Options

modules/in-commerce/units/orders/orders_event_handler.php

MINC-119 - Reset more fields during cloningClosedPublicActions