Page MenuHomeIn-Portal Phabricator
  • Changed Files
  • branches/5.2.x/core/kernel/db/dblist.php

INP-1493 - Always expand calculated fields in WHERE filters
ClosedPublic

Authored by alex on Oct 3 2015, 6:26 AM.

Details

Test Plan

Part 1

  1. create DiscountItemsEventHandler::SetCustomQuery method
  2. add WHERE filter to the ItemName calculated field (e.g. $object->addFilter('test_filter', 'ItemName LIKE ' . $this->Conn->qstr('z'));)
  3. go to E-commerceDiscounts & Certificates section
  4. create a discount with 1 product
  5. confirm, that applied before filter works as expected

Part 2

  1. create DiscountItemsEventHandler::SetCustomQuery method
  2. add HAVING filter to the ItemName calculated field (e.g. $object->addFilter('test_filter', 'ItemName LIKE ' . $this->Conn->qstr('z'), kDBList::HAVING_FILTER);)
  3. go to E-commerceDiscounts & Certificates section
  4. create a discount with 1 product
  5. confirm, that applied before filter works as expected

Diff Detail

Repository
rINP In-Portal
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 358.Oct 3 2015, 6:26 AM
alex retitled this revision from to INP-1493 - Always expand calculated fields in WHERE filters.
alex updated this object.
alex edited the test plan for this revision. (Show Details)
alex added 1 JIRA issue(s): INP-1493.
glebs accepted this revision.Oct 6 2015, 2:01 AM
glebs edited edge metadata.
This revision is now accepted and ready to land.Oct 6 2015, 2:01 AM
This revision was automatically updated to reflect the committed changes.
alex edited edge metadata.Mar 10 2016, 6:04 AM
alex added a project: Restricted Project.