Do not set primary key column before migration function is called

[jedelbo]

This is fixing a regression introduced by f7c8e97. The change done in this commit is reverted. Instead the primary key column is set in the post migration changes.

What, How & Why?

Fixes #8056

☑️ ToDos

• 📝 Changelog update
• 🚦 Tests (or not relevant)
• C-API, if public C++ API changed
• bindgen/spec.yml, if public C++ API changed

[coveralls-official]

Pull Request Test Coverage Report for Build jorgen.edelbo_478

Details

• 54 of 54 (100.0%) changed or added relevant lines in 2 files are covered.
• 47 unchanged lines in 16 files lost coverage.
• Overall coverage increased (+0.01%) to 91.11%

---

Files with Coverage Reduction	New Missed Lines	%
src/realm/dictionary.cpp	1	85.16%
src/realm/sync/network/websocket.cpp	1	72.2%
src/realm/util/compression.cpp	1	89.81%
src/realm/util/file.cpp	1	84.84%
test/test_index_string.cpp	1	93.48%
test/test_query2.cpp	1	98.73%
src/realm/table_view.cpp	2	92.99%
test/fuzz_group.cpp	2	49.28%
test/object-store/sync/flx_sync.cpp	2	98.35%
test/test_lang_bind_helper.cpp	2	93.2%

Totals
Change from base Build 2636:	0.01%
Covered Lines:	217283
Relevant Lines:	238483

---

💛 - Coveralls

[ironage]

LGTM 👍