You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficcontrol.apache.org by da...@apache.org on 2017/01/03 19:28:12 UTC

[02/13] incubator-trafficcontrol git commit: provides the ability to filter regions by division

provides the ability to filter regions by division


Project: http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/commit/a991ef92
Tree: http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/tree/a991ef92
Diff: http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/diff/a991ef92

Branch: refs/heads/master
Commit: a991ef92c0d9af4ba923605c75da21bfefdc961f
Parents: 6fc161e
Author: Jeremy Mitchell <mi...@gmail.com>
Authored: Thu Dec 29 09:36:26 2016 -0700
Committer: Jeremy Mitchell <mi...@gmail.com>
Committed: Thu Dec 29 09:36:26 2016 -0700

----------------------------------------------------------------------
 docs/source/development/traffic_ops_api/v12/region.rst |  8 ++++++++
 traffic_ops/app/lib/API/Region.pm                      | 11 +++++++++--
 2 files changed, 17 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/blob/a991ef92/docs/source/development/traffic_ops_api/v12/region.rst
----------------------------------------------------------------------
diff --git a/docs/source/development/traffic_ops_api/v12/region.rst b/docs/source/development/traffic_ops_api/v12/region.rst
index 11e2f18..74e6fbf 100644
--- a/docs/source/development/traffic_ops_api/v12/region.rst
+++ b/docs/source/development/traffic_ops_api/v12/region.rst
@@ -29,6 +29,14 @@ Regions
 
   Role(s) Required: None
 
+  **Request Route Parameters**
+
+  +---------------+----------+---------------------------------------------+
+  |   Name        | Required |                Description                  |
+  +===============+==========+=============================================+
+  |  ``division`` |    no    | Filter regions by Division ID.              |
+  +---------------+----------+---------------------------------------------+
+
   **Response Properties**
 
   +----------------------+--------+------------------------------------------------+

http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/blob/a991ef92/traffic_ops/app/lib/API/Region.pm
----------------------------------------------------------------------
diff --git a/traffic_ops/app/lib/API/Region.pm b/traffic_ops/app/lib/API/Region.pm
index a871a2b..39953d9 100644
--- a/traffic_ops/app/lib/API/Region.pm
+++ b/traffic_ops/app/lib/API/Region.pm
@@ -26,10 +26,17 @@ use MojoPlugins::Response;
 my $finfo = __FILE__ . ":";
 
 sub index {
-	my $self = shift;
+	my $self 			= shift;
+	my $division_id		= $self->param('division');
+
+	my %criteria;
+	if ( defined $division_id ) {
+		$criteria{'division'} = $division_id;
+	}
+
 	my @data;
 	my $orderby = $self->param('orderby') || "name";
-	my $rs_data = $self->db->resultset("Region")->search( undef, { prefetch => ['division'], order_by => 'me.' . $orderby } );
+	my $rs_data = $self->db->resultset("Region")->search( \%criteria, { prefetch => ['division'], order_by => 'me.' . $orderby } );
 	while ( my $row = $rs_data->next ) {
 		push(
 			@data, {