Moved GM audit calculations after configuration is checked.
This commit is contained in:
		| @@ -343,8 +343,22 @@ public class ItemTable | ||||
| 			} | ||||
| 		} | ||||
| 		 | ||||
| 		if ((actor != null) && actor.isGM()) | ||||
| 		if ((actor != null) && actor.isGM() && Config.GMAUDIT) | ||||
| 		{ | ||||
| 			final StringBuilder sb = new StringBuilder(); | ||||
| 			sb.append(process); | ||||
| 			sb.append("(id: "); | ||||
| 			sb.append(itemId); | ||||
| 			sb.append(" count: "); | ||||
| 			sb.append(count); | ||||
| 			sb.append(" name: "); | ||||
| 			sb.append(item.getItemName()); | ||||
| 			sb.append(" objId: "); | ||||
| 			sb.append(item.getObjectId()); | ||||
| 			sb.append(")"); | ||||
| 			 | ||||
| 			final String targetName = (actor.getTarget() != null ? actor.getTarget().getName() : "no-target"); | ||||
| 			 | ||||
| 			String referenceName = "no-reference"; | ||||
| 			if (reference instanceof WorldObject) | ||||
| 			{ | ||||
| @@ -354,22 +368,8 @@ public class ItemTable | ||||
| 			{ | ||||
| 				referenceName = (String) reference; | ||||
| 			} | ||||
| 			final String targetName = (actor.getTarget() != null ? actor.getTarget().getName() : "no-target"); | ||||
| 			if (Config.GMAUDIT) | ||||
| 			{ | ||||
| 				final StringBuilder sb = new StringBuilder(); | ||||
| 				sb.append(process); | ||||
| 				sb.append("(id: "); | ||||
| 				sb.append(itemId); | ||||
| 				sb.append(" count: "); | ||||
| 				sb.append(count); | ||||
| 				sb.append(" name: "); | ||||
| 				sb.append(item.getItemName()); | ||||
| 				sb.append(" objId: "); | ||||
| 				sb.append(item.getObjectId()); | ||||
| 				sb.append(")"); | ||||
| 				GMAudit.auditGMAction(actor.toString(), sb.toString(), targetName, StringUtil.concat("Object referencing this action is: ", referenceName)); | ||||
| 			} | ||||
| 			 | ||||
| 			GMAudit.auditGMAction(actor.toString(), sb.toString(), targetName, StringUtil.concat("Object referencing this action is: ", referenceName)); | ||||
| 		} | ||||
| 		 | ||||
| 		// Notify to scripts | ||||
| @@ -457,8 +457,20 @@ public class ItemTable | ||||
| 				} | ||||
| 			} | ||||
| 			 | ||||
| 			if ((actor != null) && actor.isGM()) | ||||
| 			if ((actor != null) && actor.isGM() && Config.GMAUDIT) | ||||
| 			{ | ||||
| 				final StringBuilder sb = new StringBuilder(); | ||||
| 				sb.append(process); | ||||
| 				sb.append("(id: "); | ||||
| 				sb.append(item.getId()); | ||||
| 				sb.append(" count: "); | ||||
| 				sb.append(item.getCount()); | ||||
| 				sb.append(" itemObjId: "); | ||||
| 				sb.append(item.getObjectId()); | ||||
| 				sb.append(")"); | ||||
| 				 | ||||
| 				final String targetName = (actor.getTarget() != null ? actor.getTarget().getName() : "no-target"); | ||||
| 				 | ||||
| 				String referenceName = "no-reference"; | ||||
| 				if (reference instanceof WorldObject) | ||||
| 				{ | ||||
| @@ -468,20 +480,8 @@ public class ItemTable | ||||
| 				{ | ||||
| 					referenceName = (String) reference; | ||||
| 				} | ||||
| 				final String targetName = (actor.getTarget() != null ? actor.getTarget().getName() : "no-target"); | ||||
| 				if (Config.GMAUDIT) | ||||
| 				{ | ||||
| 					final StringBuilder sb = new StringBuilder(); | ||||
| 					sb.append(process); | ||||
| 					sb.append("(id: "); | ||||
| 					sb.append(item.getId()); | ||||
| 					sb.append(" count: "); | ||||
| 					sb.append(item.getCount()); | ||||
| 					sb.append(" itemObjId: "); | ||||
| 					sb.append(item.getObjectId()); | ||||
| 					sb.append(")"); | ||||
| 					GMAudit.auditGMAction(actor.toString(), sb.toString(), targetName, StringUtil.concat("Object referencing this action is: ", referenceName)); | ||||
| 				} | ||||
| 				 | ||||
| 				GMAudit.auditGMAction(actor.toString(), sb.toString(), targetName, StringUtil.concat("Object referencing this action is: ", referenceName)); | ||||
| 			} | ||||
| 			 | ||||
| 			// if it's a pet control item, delete the pet as well | ||||
|   | ||||
| @@ -365,8 +365,18 @@ public class Item extends WorldObject | ||||
| 			} | ||||
| 		} | ||||
| 		 | ||||
| 		if ((creator != null) && creator.isGM()) | ||||
| 		if ((creator != null) && creator.isGM() && Config.GMAUDIT) | ||||
| 		{ | ||||
| 			final StringBuilder sb = new StringBuilder(); | ||||
| 			sb.append(process); | ||||
| 			sb.append("(id: "); | ||||
| 			sb.append(_itemId); | ||||
| 			sb.append(" name: "); | ||||
| 			sb.append(getName()); | ||||
| 			sb.append(")"); | ||||
| 			 | ||||
| 			final String targetName = (creator.getTarget() != null ? creator.getTarget().getName() : "no-target"); | ||||
| 			 | ||||
| 			String referenceName = "no-reference"; | ||||
| 			if (reference instanceof WorldObject) | ||||
| 			{ | ||||
| @@ -376,18 +386,8 @@ public class Item extends WorldObject | ||||
| 			{ | ||||
| 				referenceName = (String) reference; | ||||
| 			} | ||||
| 			final String targetName = (creator.getTarget() != null ? creator.getTarget().getName() : "no-target"); | ||||
| 			if (Config.GMAUDIT) | ||||
| 			{ | ||||
| 				final StringBuilder sb = new StringBuilder(); | ||||
| 				sb.append(process); | ||||
| 				sb.append("(id: "); | ||||
| 				sb.append(_itemId); | ||||
| 				sb.append(" name: "); | ||||
| 				sb.append(getName()); | ||||
| 				sb.append(")"); | ||||
| 				GMAudit.auditGMAction(creator.toString(), sb.toString(), targetName, StringUtil.concat("Object referencing this action is: ", referenceName)); | ||||
| 			} | ||||
| 			 | ||||
| 			GMAudit.auditGMAction(creator.toString(), sb.toString(), targetName, StringUtil.concat("Object referencing this action is: ", referenceName)); | ||||
| 		} | ||||
| 	} | ||||
| 	 | ||||
| @@ -562,8 +562,22 @@ public class Item extends WorldObject | ||||
| 			} | ||||
| 		} | ||||
| 		 | ||||
| 		if ((creator != null) && creator.isGM()) | ||||
| 		if ((creator != null) && creator.isGM() && Config.GMAUDIT) | ||||
| 		{ | ||||
| 			final StringBuilder sb = new StringBuilder(); | ||||
| 			sb.append(process); | ||||
| 			sb.append("(id: "); | ||||
| 			sb.append(_itemId); | ||||
| 			sb.append(" objId: "); | ||||
| 			sb.append(getObjectId()); | ||||
| 			sb.append(" name: "); | ||||
| 			sb.append(getName()); | ||||
| 			sb.append(" count: "); | ||||
| 			sb.append(count); | ||||
| 			sb.append(")"); | ||||
| 			 | ||||
| 			final String targetName = (creator.getTarget() != null ? creator.getTarget().getName() : "no-target"); | ||||
| 			 | ||||
| 			String referenceName = "no-reference"; | ||||
| 			if (reference instanceof WorldObject) | ||||
| 			{ | ||||
| @@ -573,22 +587,8 @@ public class Item extends WorldObject | ||||
| 			{ | ||||
| 				referenceName = (String) reference; | ||||
| 			} | ||||
| 			final String targetName = (creator.getTarget() != null ? creator.getTarget().getName() : "no-target"); | ||||
| 			if (Config.GMAUDIT) | ||||
| 			{ | ||||
| 				final StringBuilder sb = new StringBuilder(); | ||||
| 				sb.append(process); | ||||
| 				sb.append("(id: "); | ||||
| 				sb.append(_itemId); | ||||
| 				sb.append(" objId: "); | ||||
| 				sb.append(getObjectId()); | ||||
| 				sb.append(" name: "); | ||||
| 				sb.append(getName()); | ||||
| 				sb.append(" count: "); | ||||
| 				sb.append(count); | ||||
| 				sb.append(")"); | ||||
| 				GMAudit.auditGMAction(creator.toString(), sb.toString(), targetName, StringUtil.concat("Object referencing this action is: ", referenceName)); | ||||
| 			} | ||||
| 			 | ||||
| 			GMAudit.auditGMAction(creator.toString(), sb.toString(), targetName, StringUtil.concat("Object referencing this action is: ", referenceName)); | ||||
| 		} | ||||
| 	} | ||||
| 	 | ||||
|   | ||||
		Reference in New Issue
	
	Block a user
	 MobiusDevelopment
					MobiusDevelopment