Update names to better match existing methods

Author: Dustin McCraw

CHANGELOG.md

@@ -4,8 +4,8 @@
 - Add `divisions` endpoint.
 - Update validations on `broadcastEmail`.`saveBroadcastEmail`.
 - Add `bulkDeleteBroadcastEmails` method.
-- Add `searchDivisionMembers` method.
-- Add `searchDivisionTeams` method.
+- Add `divisionLoadMembers` method.
+- Add `divisionLoadTeams` method.
 
 ---
 

docs/collections.md

@@ -19,6 +19,7 @@ An APIv3 endpoint’s name is generally derived from its `Collection+JSON` colle
 - [Division Members](./collections/division_members.md)
 - [Division Members Preferences](./collections/division_members_preferences.md)
 - [Division Team Standings](./collections/division_team_standings.md)
+- [Divisions](./collections/divisions.md)
 - [Events](./collections/events.md)
 - [Event Statistics](./collections/event_statistics.md)
 - [Facebook Pages](./collections/facebook_pages.md)

docs/collections/broadcast_emails.md

@@ -123,7 +123,7 @@ teamsnap.saveBroadcastEmail(broadcastEmail).then(function(){
 
 <a id="bulkDeleteBroadcastEmails"></a>
 ## `bulkDeleteBroadcastEmails(broadcastEmailIds, callback)`
-Bulk deletes a list of `broadcastEmail` item.
+Bulk deletes a list of `broadcastEmail` items.
 
 ### Params
 * `broadcastEmailIds`: [array] - An array of broadcastEmailIds to be deleted.
@@ -132,7 +132,7 @@ Bulk deletes a list of `broadcastEmail` item.
 ### Examples
 ```javascript
 // ~~~~~
-// Deletes a broadcastEmail item.
+// Deletes multiple broadcastEmail items.
 teamsnap.bulkDeleteBroadcastEmails([1,2,3])
 ```
 

docs/collections/divisions.md

@@ -6,9 +6,9 @@
 - [loadDivision](#loadDivision)
 - [createDivision](#createDivision)
 - [deleteDivision](#deleteDivision)
-- [ancestorDivisions](#ancestorDivisions)
-- [descendantDivisions](#descendantDivisions)
-- [childrenDivisions](#childrenDivisions)
+- [loadAncestorDivisions](#loadAncestorDivisions)
+- [loadDescendantDivisions](#loadDescendantDivisions)
+- [loadChildDivisions](#loadChildDivisions)
 
 ---
 <a id="loadDivisions"></a>
@@ -143,54 +143,54 @@ teamsnap.saveDivision(division).then(function(){
 ---
 
 
-<a id="ancestorDivisions"></a>
-## `ancestorDivisions(divisionId, callback)`
+<a id="loadAncestorDivisions"></a>
+## `loadAncestorDivisions(divisionId, callback)`
 Returns the ancestor `division` items.
 
 ### Params
-* `divisionId`: [int] - an divisionId.
+* `divisionId`: [int] - a divisionId.
 * `callback`: [function] - callback to be executed when the operation completes.
 ### Examples
 ```javascript
 // ~~~~~
 // Returns the ancestor divisions.
-var divisions = teamsnap.ancestorDivisions(1);
+var divisions = teamsnap.loadAncestorDivisions(1);
 ```
 
 
 ---
 
 
-<a id="descendantDivisions"></a>
-## `descendantDivisions(divisionId, callback)`
+<a id="loadDescendantDivisions"></a>
+## `loadDescendantDivisions(divisionId, callback)`
 Returns the descendant `division` items.
 
 ### Params
-* `divisionId`: [int] - an divisionId.
+* `divisionId`: [int] - a divisionId.
 * `callback`: [function] - callback to be executed when the operation completes.
 ### Examples
 ```javascript
 // ~~~~~
-// Returns the ancestor divisions.
-var divisions = teamsnap.descendantDivisions(1);
+// Returns the descendant divisions.
+var divisions = teamsnap.loadDescendantDivisions(1);
 ```
 
 
 ---
 
 
-<a id="childrenDivisions"></a>
-## `childrenDivisions(divisionId, callback)`
+<a id="loadChildDivisions"></a>
+## `loadChildDivisions(divisionId, callback)`
 Returns the child `division` items.
 
 ### Params
-* `divisionId`: [int] - an divisionId.
+* `divisionId`: [int] - a divisionId.
 * `callback`: [function] - callback to be executed when the operation completes.
 ### Examples
 ```javascript
 // ~~~~~
-// Returns the ancestor divisions.
-var divisions = teamsnap.childrenDivisions(1);
+// Returns the child divisions.
+var divisions = teamsnap.loadChildDivisions(1);
 ```
 
 

docs/collections/members.md

@@ -16,7 +16,7 @@
 - [canEditItem](#canEditItem)
 - [importMembersFromTeam](#importMembersFromTeam)
 - [loadImportableMembers](#loadImportableMembers)
-- [divisionSearchMembers](#divisionSearchMembers)
+- [divisionLoadMembers](#divisionLoadMembers)
 
 
 ---
@@ -337,8 +337,8 @@ teamsnap.loadImportableMembers(8, true);
 ---
 
 
-<a id="divisionSearchMembers"></a>
-## `divisionSearchMembers(params)`
+<a id="divisionLoadMembers"></a>
+## `divisionLoadMembers(params)`
 Search for members in a division.
 
 ### Params
@@ -348,7 +348,7 @@ Search for members in a division.
 ```javascript
 // ~~~~~
 // Return all the active members in a division
-teamsnap.divisionSearchMembers({divisionId: 1, isActive: true});
+teamsnap.divisionLoadMembers({divisionId: 1, isActive: true});
 ```
 
 

docs/collections/teams.md

@@ -11,7 +11,7 @@
 - [invite](#invite)
 - [updateTimeZone](#updateTimeZone)
 - [resetStatistics](#resetStatistics)
-- [divisionSearchTeams](#divisionSearchTeams)
+- [divisionLoadTeams](#divisionLoadTeams)
 
 
 ---
@@ -251,8 +251,8 @@ teamsnap.resetStatistics(1);
 ---
 
 
-<a id="divisionSearchTeams"></a>
-## `divisionSearchTeams(params)`
+<a id="divisionLoadTeams"></a>
+## `divisionLoadTeams(params)`
 Search for teams in a division.
 
 ### Params
@@ -262,7 +262,7 @@ Search for teams in a division.
 ```javascript
 // ~~~~~
 // Return all the active teams in a division
-teamsnap.divisionSearchTeams({divisionId: 1, isActive: true});
+teamsnap.divisionLoadTeams({divisionId: 1, isActive: true});
 ```
 
 

lib/teamsnap.js

@@ -1010,27 +1010,27 @@ exports.deleteDivision = function(division, callback) {
   return this.deleteItem(division, callback);
 };
 
-exports.ancestorDivisions = function(divisionId, callback) {
+exports.loadAncestorDivisions = function(divisionId, callback) {
   if (!this.isId(divisionId)) {
-    throw new TSArgsError('teamsnap.loadDivision', 'divisionId must be provided');
+    throw new TSArgsError('teamsnap.loadAncestorDivisions', 'divisionId must be provided');
   }
   return this.collections.divisions.queryItems('ancestors', {
     id: divisionId
   }, callback);
 };
 
-exports.descendantDivisions = function(divisionId, callback) {
+exports.loadDescendantDivisions = function(divisionId, callback) {
   if (!this.isId(divisionId)) {
-    throw new TSArgsError('teamsnap.loadDivision', 'divisionId must be provided');
+    throw new TSArgsError('teamsnap.loadDescendantDivisions', 'divisionId must be provided');
   }
   return this.collections.divisions.queryItems('descendants', {
     id: divisionId
   }, callback);
 };
 
-exports.childrenDivisions = function(divisionId, callback) {
+exports.loadChildDivisions = function(divisionId, callback) {
   if (!this.isId(divisionId)) {
-    throw new TSArgsError('teamsnap.loadDivision', 'divisionId must be provided');
+    throw new TSArgsError('teamsnap.loadChildDivisions', 'divisionId must be provided');
   }
   return this.collections.divisions.queryItems('children', {
     id: divisionId
@@ -1928,9 +1928,9 @@ exports.disableMember = function(memberId, callback) {
   return this.collections.members.exec('disableMember', params).pop().callback(callback);
 };
 
-exports.divisionSearchMembers = function(params, callback) {
+exports.divisionLoadMembers = function(params, callback) {
   if (!params.divisionId) {
-    throw new TSArgsError('teamsnap.divisionSearchMembers', "`divisionId` must be provided");
+    throw new TSArgsError('teamsnap.divisionLoadMembers', "`divisionId` must be provided");
   }
   return this.collections.members.queryItems('divisionSearch', params, callback);
 };
@@ -3212,9 +3212,9 @@ exports.resetStatistics = function(teamId, callback) {
   return this.collections.teams.exec('resetStatistics', params).callback(callback);
 };
 
-exports.divisionSearchTeams = function(params, callback) {
+exports.divisionLoadTeams = function(params, callback) {
   if (!params.divisionId) {
-    throw new TSArgsError('teamsnap.divisionSearchTeams', "`divisionId` must be provided");
+    throw new TSArgsError('teamsnap.divisionLoadTeams', "`divisionId` must be provided");
   }
   return this.collections.teams.queryItems('divisionSearch', params, callback);
 };

lib/teamsnap.min.js

@@ -5322,120 +5322,11 @@ describe('Division Team Standings', function() {
 });
 
 require.register("test/divisions", function(exports, require, module) {
-describe('Divisions', function() {
-  var childDivision, newDivision, rootDivision;
-  newDivision = null;
-  rootDivision = null;
-  childDivision = null;
-  before(function(done) {
-    rootDivision = teamsnap.createDivision({
-      name: 'Root Division',
-      sportId: 1,
-      country: 'United States',
-      postalCode: 80302,
-      timeZone: 'America/Denver',
-      parentId: 1
-    });
-    return teamsnap.saveDivision(rootDivision, function(err, result) {
-      expect(err).to.be["null"];
-      childDivision = teamsnap.createDivision({
-        name: 'Child Division',
-        sportId: 1,
-        parentId: rootDivision.id,
-        country: 'United States',
-        postalCode: 80302,
-        timeZone: 'America/Denver',
-        parentId: rootDivision.id
-      });
-      return teamsnap.saveDivision(childDivision, function(err, result) {
-        expect(err).to.be["null"];
-        return done();
-      });
-    });
-  });
-  after(function(done) {
-    return teamsnap.deleteDivision(childDivision, function(err, result) {
-      expect(err).to.be["null"];
-      return done();
-    });
-  });
-  after(function(done) {
-    if (newDivision) {
-      return teamsnap.deleteDivision(newDivision, function(err, result) {
-        expect(err).to.be["null"];
-        return done();
-      });
-    } else {
-      return done();
-    }
-  });
-  after(function(done) {
-    return teamsnap.deleteDivision(rootDivision, function(err, result) {
-      expect(err).to.be["null"];
-      return done();
-    });
-  });
-  it('should be able to create a new division', function(done) {
-    newDivision = teamsnap.createDivision({
-      name: 'New Test Division',
-      sportId: 1,
-      country: 'United States',
-      postalCode: 80302,
-      timeZone: 'America/Denver',
-      parentId: rootDivision.id
-    });
-    return teamsnap.saveDivision(newDivision, function(err, result) {
-      expect(err).to.be["null"];
-      return done();
-    });
-  });
-  it('should be able to load all divisions', function(done) {
-    return teamsnap.loadDivisions(function(err, result) {
-      expect(err).to.be["null"];
-      result.should.be.an('array');
-      return done();
-    });
-  });
-  it('should be able to load a division', function(done) {
-    return teamsnap.loadDivision(rootDivision.id, function(err, result) {
-      expect(err).to.be["null"];
-      result.should.be.an('object');
-      return done();
-    });
-  });
-  it('should be able to delete a division', function(done) {
-    return teamsnap.deleteDivision(newDivision, function(err, result) {
-      expect(err).to.be["null"];
-      newDivision = null;
-      return done();
-    });
-  });
-  it('should be able to return a divisions ancestors', function(done) {
-    return teamsnap.ancestorDivisions(childDivision.id, function(err, result) {
-      expect(err).to.be["null"];
-      result.should.be.an('array');
-      return done();
-    });
-  });
-  it('should be able to return a divisions descendants', function(done) {
-    return teamsnap.descendantDivisions(rootDivision.id, function(err, result) {
-      expect(err).to.be["null"];
-      result.should.be.an('array');
-      return done();
-    });
-  });
-  return it('should be able to return a divisions children', function(done) {
-    return teamsnap.childrenDivisions(rootDivision.id, function(err, result) {
-      expect(err).to.be["null"];
-      result.should.be.an('array');
-      return done();
-    });
-  });
-});
+
 
 });
 
-require.register("test/eventStatistics", function(exports, require, module) {
+;require.register("test/eventStatistics", function(exports, require, module) {
 describe('Event Statistics', function() {
   return it('should be able to load all event statistics for a team', function(done) {
     return teamsnap.loadEventStatistics(team.id, function(err, result) {