Page MenuHomeIn-Portal Phabricator

INP-1798 - Include validation errors and temp mode flag in kDBItem validation error message
ClosedPublic

Authored by alex on Mar 4 2021, 11:31 AM.

Details

Test Plan
  1. login to Admin Console
  2. go to Website & ContentLabels & Phrases section
  3. press Add toolbar button in the grid, that appeared
  4. press Save toolbar button in the popup window, that appeared
  5. open Debugger by pressing Show Debugger button in Debugger Toolbar in that popup/modal window
  6. confirm, that instead of 2 records (Validation failed ... and actual validation errors) now there is single message with all that data
  7. confirm, that temp table usage fact is also present in validation error message
  8. close Debugger panel and editing window
  9. go to Logs & ReportsSystem Log section
  10. open system log record, that is matching validation error shown in Debugger panel previously
  11. confirm, that it's text (log message itself) is matching to what was displayed in the Debugger panel previously

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 created this revision.Mar 4 2021, 11:31 AM
alex requested review of this revision.Mar 4 2021, 11:31 AM
alex added a project: Restricted Project.Mar 4 2021, 11:32 AM
erik accepted this revision.Mar 8 2021, 4:42 AM
This revision is now accepted and ready to land.Mar 8 2021, 4:42 AM
This revision was automatically updated to reflect the committed changes.