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

INP-1492 - Replace "%2$s" with current language in "kDBList::extractCalculatedFields" method
ClosedPublic

Authored by alex on Oct 3 2015, 5:39 AM.

Details

Test Plan
  • in PhpStorm:
    1. comment out the di:OnAfterConfigRead event
    2. change ItemName calculated field (of di unit) declaration into 'IF(p.Name IS NULL,c.l%2$s_Name,p.l%2$s_Name)'
  • in Admin Console:
    1. go to E-commerceDiscounts & Certificates section
    2. create discount, that has 1 product
    3. confirm, that product name is displayed correctly on Items tab during discount editing
    4. use Item Name column filter to search for letter present in product name
    5. confirm, that product is displayed and no SQL error present in Debugger
    6. use Item Name column filter to search for letter not present in product name
    7. confirm, that product isn't displayed and no SQL error present in Debugger
    8. remove grid search filter
  • in PhpStorm:
    1. create the DiscountItemsEventHandler::SetCustomQuery method in which apply HAVING filter to the ItemName column (e.g. $object->addFilter('test_filter', 'ItemName LIKE ' . $this->Conn->qstr('z'), kDBList::HAVING_FILTER);)
    2. confirm, that no SQL error happens and data is filtered 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 357.Oct 3 2015, 5:39 AM
alex retitled this revision from to INP-1492 - Replace "%2$s" with current language in "kDBList::extractCalculatedFields" method.
alex updated this object.
alex edited the test plan for this revision. (Show Details)
alex added 1 JIRA issue(s): INP-1492.
glebs accepted this revision.Oct 6 2015, 1:55 AM
glebs edited edge metadata.
This revision is now accepted and ready to land.Oct 6 2015, 1:55 AM
This revision was automatically updated to reflect the committed changes.
alex edited edge metadata.Mar 10 2016, 5:42 AM
alex added a project: Restricted Project.