…en this function is called multiple times from negotiate.go line 252, this prevents adding values multiple times.

…tract and not a renewal

… new contract and not a renewal"

This reverts commit 9d08da8.

Contract counter should be increased as it is forming a new obligation
for new contracts as well for renewals.

…tion. When this function is called multiple times from negotiate.go line 252, this prevents adding values multiple times."

This reverts commit e15c6d8.

When there is an error with the queuing of the action items, the storage obligation will be removed with status 'obligationRejected' and the function will return. Placing the financial metrics update at the end of the function will
in this case never be reached.

there is nothing to revert.

…ror.

Financial metrics are only updated when there are no errors.

AcceptTransactionSet call returns with an error.

This should reset the host to the original state. However, the rejected storage
obligation will stay in the database as rejected without SectorRoots.