Peerplays_Blockchain/peerplays_migrated
1
0
Fork 0
Code Issues 35 Pull requests 4 Projects Releases 20 Packages Wiki Activity Actions

Fix for undo crash

Browse source
This commit is contained in:
Davor Hirunda 2022-11-15 22:34:05 +00:00 committed by serkixenos
parent d5d6390030
commit 811d68ef4d
3 changed files with 30 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
libraries/chain/hardfork.d/SIDECHAIN.hf Normal file
View file

@ -0,0 +1,7 @@
#ifndef HARDFORK_SIDECHAIN_DELETE_TIME
#ifdef BUILD_PEERPLAYS_TESTNET
#define HARDFORK_SIDECHAIN_DELETE_TIME (fc::time_point_sec::from_iso_string("2022-11-16T02:00:00"))
#else
#define HARDFORK_SIDECHAIN_DELETE_TIME (fc::time_point_sec::from_iso_string("2022-11-16T02:00:00"))
#endif
#endif

27
libraries/chain/sidechain_address_evaluator.cpp
View file

@ -22,12 +22,14 @@ object_id_type add_sidechain_address_evaluator::do_apply(const sidechain_address
const auto &sidechain_addresses_idx = db().get_index_type<sidechain_address_index>().indices().get<by_account_and_sidechain_and_expires>(); const auto &sidechain_addresses_idx = db().get_index_type<sidechain_address_index>().indices().get<by_account_and_sidechain_and_expires>();
const auto &addr_itr = sidechain_addresses_idx.find(std::make_tuple(op.sidechain_address_account, op.sidechain, time_point_sec::maximum())); const auto &addr_itr = sidechain_addresses_idx.find(std::make_tuple(op.sidechain_address_account, op.sidechain, time_point_sec::maximum()));
if (addr_itr != sidechain_addresses_idx.end()) if (addr_itr != sidechain_addresses_idx.end()) {
{ if (db().head_block_time() >= HARDFORK_SIDECHAIN_DELETE_TIME) {
//db().modify(*addr_itr, [&](sidechain_address_object &sao) { db().remove(*addr_itr);
// sao.expires = db().head_block_time(); } else {
//}); db().modify(*addr_itr, [&](sidechain_address_object &sao) {
db().remove(*addr_itr); sao.expires = db().head_block_time();
});
}
} }
const auto& new_sidechain_address_object = db().create<sidechain_address_object>( [&]( sidechain_address_object& obj ){ const auto& new_sidechain_address_object = db().create<sidechain_address_object>( [&]( sidechain_address_object& obj ){
@ -106,11 +108,14 @@ void_result delete_sidechain_address_evaluator::do_apply(const sidechain_address
{ try { { try {
const auto& idx = db().get_index_type<sidechain_address_index>().indices().get<by_id>(); const auto& idx = db().get_index_type<sidechain_address_index>().indices().get<by_id>();
auto sidechain_address = idx.find(op.sidechain_address_id); auto sidechain_address = idx.find(op.sidechain_address_id);
if(sidechain_address != idx.end()) { if (sidechain_address != idx.end()) {
//db().modify(*sidechain_address, [&](sidechain_address_object &sao) { if (db().head_block_time() >= HARDFORK_SIDECHAIN_DELETE_TIME) {
// sao.expires = db().head_block_time(); db().remove(*sidechain_address);
//}); } else {
db().remove(*sidechain_address); db().modify(*sidechain_address, [&](sidechain_address_object &sao) {
sao.expires = db().head_block_time();
});
}
} }
return void_result(); return void_result();
} FC_CAPTURE_AND_RETHROW( (op) ) } } FC_CAPTURE_AND_RETHROW( (op) ) }

14
tests/tests/sidechain_addresses_test.cpp
View file

@ -244,6 +244,9 @@ BOOST_AUTO_TEST_CASE( sidechain_address_delete_test ) {
BOOST_TEST_MESSAGE("sidechain_address_delete_test"); BOOST_TEST_MESSAGE("sidechain_address_delete_test");
generate_blocks(HARDFORK_SIDECHAIN_DELETE_TIME);
generate_block();
INVOKE(sidechain_address_add_test); INVOKE(sidechain_address_add_test);
GET_ACTOR(alice); GET_ACTOR(alice);
@ -266,18 +269,12 @@ BOOST_AUTO_TEST_CASE( sidechain_address_delete_test ) {
sign(trx, alice_private_key); sign(trx, alice_private_key);
PUSH_TX(db, trx, ~0); PUSH_TX(db, trx, ~0);
} }
//time_point_sec now = db.head_block_time();
generate_block();
generate_block();
{ {
BOOST_TEST_MESSAGE("Check sidechain_address_delete_operation results"); BOOST_TEST_MESSAGE("Check sidechain_address_delete_operation results");
const auto& idx = db.get_index_type<sidechain_address_index>().indices().get<by_account_and_sidechain_and_expires>(); const auto& idx = db.get_index_type<sidechain_address_index>().indices().get<by_account_and_sidechain_and_expires>();
//BOOST_REQUIRE( idx.size() == 1 );
//auto obj = idx.find( boost::make_tuple( alice_id, sidechain_type::bitcoin, time_point_sec::maximum() ) );
//BOOST_REQUIRE( obj == idx.end() );
//auto expired_obj = idx.find( boost::make_tuple( alice_id, sidechain_type::bitcoin, now ) );
//BOOST_REQUIRE( expired_obj != idx.end() );
BOOST_REQUIRE( idx.size() == 0 ); BOOST_REQUIRE( idx.size() == 0 );
} }
} }
@ -286,6 +283,9 @@ BOOST_AUTO_TEST_CASE(sidechain_address_delete_create_test) {
BOOST_TEST_MESSAGE("sidechain_address_delete_create_test"); BOOST_TEST_MESSAGE("sidechain_address_delete_create_test");
generate_blocks(HARDFORK_SIDECHAIN_DELETE_TIME);
generate_block();
INVOKE(sidechain_address_add_test); INVOKE(sidechain_address_add_test);
GET_ACTOR(alice); GET_ACTOR(alice);