Comparators no longer work as expected reading containers through powered blocks

請只寫英文。僅添加有用的信息。我們知道這是影響Redstone的變化。請保持冷靜。謝謝。

Please write only English. Only add useful information. We know this is a change that affects Redstone. Please stay calm. Thank you.

Confirmed for 20w09a

I'm not redstone player but I understand these players anxiety about this change. Still, please speak English in this forum.

Please take further discussion of this ticket to the Mojira subreddit. This ticket has already been assigned and is likely to be fixed, spamming the comments won't make it get fixed any faster.

Confirmed for 20w09a

On a side note, the feature related to this bug has been first reported in MC-64394, which (the feature) has been resolved as working as intended.

Some code analysis:
Old logic (FabricMC yarn 1.15-pre4 build 1)

protected int getPower(World world, BlockPos pos, BlockState state) {
      int i = super.getPower(world, pos, state);
      Direction direction = (Direction)state.get(FACING);
      BlockPos blockPos = pos.offset(direction);
      BlockState blockState = world.getBlockState(blockPos);
      if (blockState.hasComparatorOutput()) {
         i = blockState.getComparatorOutput(world, blockPos);
      } else if (i < 15 && blockState.isSimpleFullBlock(world, blockPos)) {
         blockPos = blockPos.offset(direction);
         blockState = world.getBlockState(blockPos);
         if (blockState.hasComparatorOutput()) {
            i = blockState.getComparatorOutput(world, blockPos);
         } else if (blockState.isAir()) {
            ItemFrameEntity itemFrameEntity = this.getAttachedItemFrame(world, direction, blockPos);
            if (itemFrameEntity != null) {
               i = itemFrameEntity.getComparatorPower();
            }
         }
      }

      return i;
   }

New logic (FabricMC yarn 20w08a build 6) with my comments:

protected int getPower(World world, BlockPos pos, BlockState state) {
      int i = super.getPower(world, pos, state);
      Direction direction = (Direction)state.get(FACING);
      BlockPos blockPos = pos.offset(direction);
      BlockState blockState = world.getBlockState(blockPos);
      if (blockState.hasComparatorOutput()) {
         i = blockState.getComparatorOutput(world, blockPos);
      } else if (i < 15 && blockState.isSimpleFullBlock(world, blockPos)) {
         blockPos = blockPos.offset(direction);
         blockState = world.getBlockState(blockPos);
         ItemFrameEntity itemFrameEntity = this.getAttachedItemFrame(world, direction, blockPos); // also the air check is now removed, most likely intended
         int j = itemFrameEntity == null ? 0 : itemFrameEntity.getComparatorPower();
         int k = blockState.hasComparatorOutput() ? blockState.getComparatorOutput(world, blockPos) : 0;
         i = Math.max(Math.max(j, k), i); // i = Math.max(j, k) may be a fix, but it still changes old behavior as previously only frames in air outputs, and has lower priority than outputing blocks
      }

      return i;
   }

The max out calculation is all new. The old logic is that it replaces the power obtained as a regular block with custom comparator output if i < 15.

Is maxing things out wrong?

This i < 15 statement makes me think that maxing out power later is the intended initial behavior (given if i == 15, extra check will not return a higher value and will waste cpu/memory when we are actually doing maxing out logic).

Is maxing things out right?

If it is, the getting power not through wall code probably should max with ambient block power as well, but it's clear that if the comparator outputing block has redstone output itself, things are clearly messed up.

My conclusion is that the final result is up to mojang to decide. In my opinion, the current status (i.e. in 20w09a) should be the most correct one (again this is only my opinion). The other side effects of the recent change (item frame) may be less noticed. If we do determine what should be regarded as "correct", we should determine what should be the power when both item frame and a comparator outputing block exists.