The view value is misused

[majecty]

A block author writes view data in the block's seal. The view value specifies when the author created the block. Other nodes use the view field to verify whether the block author is valid or not.

When a block is finalized, the next height's proposer writes the finalized view in the next block. There are signatures in a block. A signature is a precommit message for the previous block, and the message of the signature contains the finalized view. Thus, validators use the finalized view value to check the signature.

Hence, there are two values for a block. One is the "created" view that it is saved in the block, and the other one is the "finalized" that is saved in the next block.

Most of the time, the two view value will be identical. However, if there are network issues, the two values can differ. The current code assumes that the two values are the same. We should fix the code.

[majecty]

Fixed by #1820