Page MenuHomePhabricator
Differential D17520

Add better error checking to 'Add to Dashboard'
ClosedPublic
Actions

Authored by chad on Mar 20 2017, 9:47 PM.
Tags
None
Referenced Files
F19941083: D17520.diff
Wed, Apr 15, 7:44 AM
F19937183: D17520.id42131.diff
Sun, Apr 12, 7:56 PM
F19863066: D17520.id42132.diff
Mar 14 2026, 11:09 AM
F19629668: D17520.id42131.diff
Feb 5 2026, 12:20 AM
F19433381: D17520.id42132.diff
Dec 28 2025, 1:56 PM
F19283233: D17520.id.diff
Dec 23 2025, 4:38 AM
F19279716: D17520.id42131.diff
Dec 23 2025, 2:56 AM
F19266293: D17520.diff
Dec 22 2025, 7:57 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T5307: UX for taking actions with search results
Commits
rP9b07adb8dad3: Add better error checking to 'Add to Dashboard'
Summary

Ref T5307. Adds a better query check query, sets required for the name, adds the correct URI for cancelling.

Test Plan

Test a form without a name, fake a query string, test cancel button.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad created this revision.Mar 20 2017, 9:47 PM
Herald added a subscriber: Korvin. · View Herald TranscriptMar 20 2017, 9:47 PM
Harbormaster completed remote builds in B16046: Diff 42131.Mar 20 2017, 9:48 PM
epriestley accepted this revision.Mar 20 2017, 9:52 PM
epriestley added inline comments.
src/applications/dashboard/controller/PhabricatorDashboardQueryPanelInstallController.php
44

who's a good query

is it you? are you a good query??

pet pet pat yes you're a good query

This revision is now accepted and ready to land.Mar 20 2017, 9:52 PM
chad added a comment.Mar 20 2017, 9:55 PM

I'm not a fan of adding the Dashboard ID to select, but maybe I could just switch to typeahead if you have more than n dashboards. I just want to solve a real problem not a test data problem. Will see what happens when I use it on this install. :)

Closed by commit rP9b07adb8dad3: Add better error checking to 'Add to Dashboard' (authored by chad, committed by chad). · Explain WhyMar 20 2017, 9:55 PM
This revision was automatically updated to reflect the committed changes.
chad added a comment.Mar 20 2017, 10:01 PM

404

https://secure.phabricator.com/dashboard/panel/install/DifferentialRevisionSearchEngine/authored/

epriestley added inline comments.Mar 20 2017, 10:06 PM
src/applications/dashboard/controller/PhabricatorDashboardQueryPanelInstallController.php
32

Oh, this should be $v_query.

Revision Contents
Changeset List

PathSize
src/
applications/
dashboard/
controller/
24 lines

Diff 42132

View Options

src/applications/dashboard/controller/PhabricatorDashboardQueryPanelInstallController.php

Loading...
Phabricator · Built by Phacility