[I-02] Commented Out Code in 'certified' Function

[softstackio]

Likelihood: Low

Description
In the 'certified' function of the CertifierHbbft contract, there is a commented out section that references a 'stakingContract'. This commented code suggests an incomplete or planned functionality related to the staking contract. The presence of commented code in a production contract can lead to confusion about the intended behaviour and may indicate unfinished or abandoned features.

The commented section is as follows:

// IStakingHbbft stakingContract = IStakingHbbft(
//     validatorSetContract.getStakingContract()
// );

Recommendation:

1. Review the purpose of the commented code and determine if it's needed for future functionality.
2. If the code is no longer relevant, remove it entirely to improve code clarity.
3. If the functionality is intended for future use, consider adding a TODO comment
   explaining the plan for this code.
4. If the functionality should be implemented, uncomment the code, complete the
   implementation, and thoroughly test it.
5. Ensure all code in production contracts is active and serves a purpose to avoid
   confusion and potential security risks.