Differential D131

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

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

Tags

Subscribers

None

Details

Reviewers

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

rMINC Modules.In-Commerce

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

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

Diff 381

branches/5.2.x/units/orders/orders_event_handler.php

Loading...