Differential D131

MINC-133 - Don't run "OrdersEventHandler::CheckUser" method in CRON
ClosedPublic
Actions

Authored by alex on Sep 18 2015, 11:03 AM.

Details

Reviewers

glebs

Commits

rMINC16289: Fixes MINC-133 - Don't run "OrdersEventHandler::CheckUser" method in CRON

Test Plan

create an order on Front-End with 1 product while being logged-in
create event in ord unit, that would take that order, load it, update it without changing any fields
set above created event as scheduled task to be executed once per hour
run php tools/cron.php
confirm, that customer in the order wasn't changed

Diff Detail

Repository

rINP In-Portal

Branch

/in-portal/branches/5.2.x

Lint

Lint OK

Unit

No Unit Test Coverage

Build Status

Buildable 145
Build 145: arc lint + arc unit

Event Timeline

alex updated this revision to Diff 308.Sep 18 2015, 11:03 AM

alex retitled this revision from to MINC-133 - Don't run "OrdersEventHandler::CheckUser" method in CRON.

alex updated this object.

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

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

Herald added a reviewer: glebs. · View Herald TranscriptSep 18 2015, 11:03 AM

glebs accepted this revision.Sep 21 2015, 1:03 AM

glebs edited edge metadata.

This revision is now accepted and ready to land.Sep 21 2015, 1:03 AM

Closed by commit rMINC16289: Fixes MINC-133 - Don't run "OrdersEventHandler::CheckUser" method in CRON (authored by alex). · Explain WhyOct 25 2015, 5:56 AM

This revision was automatically updated to reflect the committed changes.

alex edited edge metadata.Mar 10 2016, 5:43 AM

alex added a project: Restricted Project.

Revision Contents
Changeset List

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

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	308	16244		Sep 18 2015, 11:03 AM	★	★
Diff 2	381	16287	Commit rMINC16289	Oct 25 2015, 5:56 AM	★	★

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

Diff 308

View Options

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

MINC-133 - Don't run "OrdersEventHandler::CheckUser" method in CRONClosedPublicActions