realm / realm-core

Core database component for the Realm Mobile Database SDKs
https://realm.io
Apache License 2.0
1.02k stars 165 forks source link

RCORE-2265: Fix issue when changing type of primary key #8046

Closed jedelbo closed 1 week ago

jedelbo commented 2 weeks ago

If you change the type of the primary key and call Realm::update_schema() without a migration function, the new column will not be set as primary key column.

What, How & Why?

Fixes #8045

☑️ ToDos

coveralls-official[bot] commented 2 weeks ago

Pull Request Test Coverage Report for Build jorgen.edelbo_455

Details


Files with Coverage Reduction New Missed Lines %
src/realm/dictionary.cpp 1 85.16%
src/realm/query_expression.hpp 1 93.81%
src/realm/util/compression.cpp 1 89.81%
src/realm/uuid.cpp 1 98.48%
src/realm/array_blobs_big.cpp 2 98.58%
src/realm/cluster.cpp 2 75.85%
src/realm/object-store/shared_realm.cpp 2 91.9%
src/realm/sync/noinst/client_impl_base.cpp 3 82.78%
src/realm/sync/noinst/protocol_codec.hpp 3 75.74%
src/realm/table.cpp 3 90.59%
<!-- Total: 70 -->
Totals Coverage Status
Change from base Build 2629: 0.01%
Covered Lines: 217269
Relevant Lines: 238434

💛 - Coveralls