X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=sharedutilities.cpp;h=71d7782cad87c7914c2d14071ec1cc657b2803d1;hb=3c5a709812a966bf3cabc8bb15f86f0c36cf1e34;hp=15f16a6f1942921ca41098dd502750d68c75ce3e;hpb=ca9ac1d80c62f57270b0dcd49410ebe08a8aecd6;p=mothur.git

diff --git a/sharedutilities.cpp b/sharedutilities.cpp
index 15f16a6..71d7782 100644
--- a/sharedutilities.cpp
+++ b/sharedutilities.cpp
@@ -102,6 +102,10 @@ void SharedUtil::getSharedVectorswithReplacement(vector<string> Groups, vector<S
 //need to have mode because different commands require different number of valid groups
 void SharedUtil::setGroups(vector<string>& userGroups, vector<string>& allGroups) {
 	try {
+		
+		sort(userGroups.begin(), userGroups.end());
+		sort(allGroups.begin(), allGroups.end());
+		
 		if (userGroups.size() != 0) {
 			if (userGroups[0] != "all") {
 				//check that groups are valid
@@ -116,7 +120,7 @@ void SharedUtil::setGroups(vector<string>& userGroups, vector<string>& allGroups
 				
 				//if the user only entered invalid groups
 				if (userGroups.size() == 0) { 
-					m->mothurOut("You provided no valid groups. I will run the command using all the groups in your groupfile."); m->mothurOutEndLine();
+					m->mothurOut("You provided no valid groups. I will run the command using all the groups in your file."); m->mothurOutEndLine();
 					for (int i = 0; i < allGroups.size(); i++) {
 						userGroups.push_back(allGroups[i]);
 					}
@@ -144,6 +148,10 @@ void SharedUtil::setGroups(vector<string>& userGroups, vector<string>& allGroups
 //need to have mode because different commands require different number of valid groups
 void SharedUtil::setGroups(vector<string>& userGroups, vector<string>& allGroups, string mode) {
 	try {
+		
+		sort(userGroups.begin(), userGroups.end());
+		sort(allGroups.begin(), allGroups.end());
+		
 		if (userGroups.size() != 0) {
 			if (userGroups[0] != "all") {
 				//check that groups are valid
@@ -190,6 +198,9 @@ void SharedUtil::setGroups(vector<string>& userGroups, vector<string>& allGroups
 //for parsimony and unifrac commands you set pairwise groups as well as an allgroups in calc
 void SharedUtil::setGroups(vector<string>& userGroups, vector<string>& allGroups, string& label, int& numGroups, string mode){  //globaldata->Groups, your tree or group map, allgroups, mode
 	try {
+		sort(userGroups.begin(), userGroups.end());
+		sort(allGroups.begin(), allGroups.end());
+		
 		numGroups = 0;
 		label = "";
 
@@ -271,6 +282,7 @@ void SharedUtil::setGroups(vector<string>& userGroups, vector<string>& allGroups
 /**************************************************************************************/
 void SharedUtil::getCombos(vector<string>& groupComb, vector<string> userGroups, int& numComp) { //groupcomb, globaldata->Groups, numcomb
 	try {
+		sort(userGroups.begin(), userGroups.end());
 		//calculate number of comparisons i.e. with groups A,B,C = AB, AC, BC = 3;
 		numComp = 0;
 		for (int i=0; i< userGroups.size(); i++) { 
@@ -305,7 +317,7 @@ bool SharedUtil::isValidGroup(string groupname, vector<string> groups) {
 	}
 }
 
-/**************************************************************************************
+/**************************************************************************************/
 void SharedUtil::updateGroupIndex(vector<string>& userGroups, map<string, int>& index) {
 	try {
 		index.clear();