Minecraft-Java-Edition-Commands – Need Help with a Specific Minecraft Testfor Command

minecraft-commandsminecraft-java-edition

Im trying to use commands blocks, scoreboards, and redstone to give all my moderators effects when they run out. I already have the system to reapply the effects if you are in the team working but now I need to test for both being on the team moderators and not having the effect in one command block.

I tried this initially:

/testfor @a {ActiveEffects:[{Id:21b,Amplifier:20b}],Team=Moderators}

But I got an error in the output of the command block:

[17:45:02] Data tag parsing failed: Unable to locate name/value separator for string: Team=Moderators

Now please keep in mind I am not in 1.9 on my server yet meaning I do not have access to any commands related to that version.
I believe this concept could be used on other servers that don't want to create their own new plugins for this and may rather use command blocks such as I have.

Best Answer

Fixing your command

Your problem here is that you are mixing up target selector arguments with data tags, and then some minor mistakes.

Target selector arguments, such as team=Moderators (lowercase!) are placed in square brackets ([]) directly with the target selector, such as @a or @p. They come in the form key=value, with a predefined set of valid keys.

Data tag matching on the other hand is looking directly at the NBT data of an entity, such as ActiveEffects. They are placed within curly brackets ({}) and come in the form of key:value. Only a fairly small subset of commands are capable of using data tags at all.

Your command in this case should be

/testfor @a[team=Moderators] {ActiveEffects:[{Id:21b,Amplifier:20b}]}

Fixing your setup

However, your whole idea is rubbish, if I may be so blunt. It looks like you are testing if there are moderators with Health Boost 20, then invert the signal and re-apply health boost. The first problem is that no moderator gets health boost unless none of the moderators currently have health boost. Actually negating data tag matching requires assigning scoreboard values based on the data tag.

For most effects, reapplying the effect all the time works just fine, e.g. run

/effect @a[team=Moderators] <effect> 1 <amplifier> true

on a fast clock (setblock/fill clock!) works just fine, and is actually way less resource-intensive than a redstone lag-machine with torches, comparators and whatnot.

However, there is an issue with reapplying Health Boost since it will reset your current health to 10 hearts all the time. If you want to reapply the effect only when needed, using /testfor, comparators and redstone is still not a good idea. Instead, you should translate the data tag into a scoreboard value which can be used in a target selector. To do that, start by creating a scoreboard objective:

/scoreboard objectives add hasEffect dummy

Create a fill/setblock clock and put the following commands in order:

/scoreboard players set @a[team=Moderators] hasEffect 0
/scoreboard players set @a[team=Moderators] hasEffect 1 {ActiveEffects:[{Id:21b,Amplifier:20b}]}
/effect @a[team=Moderators,score_hasEffects=0] minecraft:health_boost 9999 20 true

Related Solutions

Minecraft Java Edition – How to Mark or Modify a Player Using Inventory Items in Minecraft Commands

As of 1.13, doing this no longer needs a scoreboard and can be done in one command block:

/execute as @a[nbt={Inventory:[{id:"blue_banner"}]}] run ...

Minecraft – How to replace a SPECIFIC item in a players hotbar with another specific item

You'll need to "translate" the glass bottle data tag into something that works with target selectors (@a and so on). This can be achieved most easily using dummy scoreboard values or, in 1.9, scoreboard tags.

In 1.8, create an objective using

scoreboard objectives add bottleInOne dummy

Create a fill clock and run the following two commands

scoreboard players set @a[score_bottleInOne_min=1] bottleInOne 0
scoreboard players set @a[score_bottleInOne=0] bottleInOne 1 {SelectedItemSlot:0,SelectedItem:{id:minecraft:glass_bottle}}

If the team part is necessary, you can now add @a[score_bottleInOne_min=1] to the team, and remove @a[score_bottleInOne=0] from it, using the same fill clock. You could also use these target selectors directly for the /effect and /replaceitem commands, if the team part was just your idea of selecting that player for the effect.

In 1.9, you can omit the scoreboard objective in favor of tags. Using a Repeat/Chain command block chain, run

scoreboard players tag @a[tag=bottleInOne] remove bottleInOne
scoreboard players tag @a[tag=!bottleInOne] add bottleInOne {SelectedItemSlot:0,SelectedItem:{id:minecraft:glass_bottle}}

In the following commands, you can then use @a[tag=bottleInOne] or @a[tag=!bottleInOne] to target players with and without the bottle selected in slot 0, respectively.