use height to hash in short sync

[almogdepaz]

Purpose:

dont use the cache to locate the forkpoint

Current Behavior:

we use the block records cache to determine fork point even though the cache may contain orphan blocks, this can cause us to use a wrong fork point while syncing

New Behavior:

use height to hash which is more correct

Testing Notes:

[coveralls-official]

Pull Request Test Coverage Report for Build 12275663924

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 8 of 8 (100.0%) changed or added relevant lines in 1 file are covered.
• 244 unchanged lines in 14 files lost coverage.
• Overall coverage increased (+0.07%) to 91.537%

---

Files with Coverage Reduction	New Missed Lines	%
chia/_tests/wallet/test_transaction_store.py	1	99.58%
chia/_tests/core/full_node/stores/test_full_node_store.py	1	99.5%
chia/_tests/core/full_node/stores/test_block_store.py	2	99.53%
chia/_tests/pools/test_pool_rpc.py	3	99.07%
chia/wallet/transaction_record.py	3	97.74%
chia/wallet/wallet_transaction_store.py	5	97.9%
chia/_tests/blockchain/test_blockchain.py	9	99.26%
chia/_tests/core/full_node/test_full_node.py	10	98.62%
chia/_tests/wallet/sync/test_wallet_sync.py	12	98.63%
chia/_tests/core/mempool/test_mempool.py	13	98.75%

Totals
Change from base Build 12209661102:	0.07%
Covered Lines:	104661
Relevant Lines:	114156

---

💛 - Coveralls