diff --git a/include/caosdb/transaction_status.h b/include/caosdb/transaction_status.h index 8645aceb3749efbbcf8c6cfec5aa96394b029ed7..e7a7b3ec25eb7ab97368a3b90507490551d717da 100644 --- a/include/caosdb/transaction_status.h +++ b/include/caosdb/transaction_status.h @@ -43,6 +43,8 @@ using caosdb::exceptions::TransactionError; */ class TransactionStatus { public: + // REFACTORING NEEDED: When you touch this code again consider writing a + // macro, because this is a lot of redundant code here... inline static auto INITIAL() -> const TransactionStatus & { static const TransactionStatus initial( StatusCode::INITIAL, caosdb::get_status_description(StatusCode::INITIAL)); @@ -61,6 +63,9 @@ public: } inline static auto RPC_ERROR(const std::string &details) -> const TransactionStatus { + // We use the GENERIC_RPC_ERROR here because we might want to add further + // RPC_ERROR states with different error codes (which stem from GRPC) here + // in the future. return TransactionStatus( StatusCode::GENERIC_RPC_ERROR, caosdb::get_status_description(StatusCode::GENERIC_RPC_ERROR) +