Skip to content

Commit 06dc612

Browse files
committed
CXX-1053 database::create_collection(): call options methods in order
1 parent b3d9938 commit 06dc612

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

src/mongocxx/database.cpp

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -115,34 +115,34 @@ class collection database::create_collection(bsoncxx::string::view_or_value name
115115
const options::create_collection& options) {
116116
document options_builder{};
117117

118-
if (options.capped()) {
119-
options_builder << "capped" << *options.capped();
120-
}
121-
122118
if (options.auto_index_id()) {
123119
options_builder << "autoIndexId" << *options.auto_index_id();
124120
}
125121

126-
if (options.size()) {
127-
options_builder << "size" << *options.size();
128-
}
129-
130-
if (options.max()) {
131-
options_builder << "max" << *options.max();
122+
if (options.capped()) {
123+
options_builder << "capped" << *options.capped();
132124
}
133125

134126
if (options.collation()) {
135127
options_builder << "collation" << *options.collation();
136128
}
137129

138-
if (options.storage_engine()) {
139-
options_builder << "storageEngine" << *options.storage_engine();
130+
if (options.max()) {
131+
options_builder << "max" << *options.max();
140132
}
141133

142134
if (options.no_padding()) {
143135
options_builder << "flags" << (*options.no_padding() ? 0x10 : 0x00);
144136
}
145137

138+
if (options.size()) {
139+
options_builder << "size" << *options.size();
140+
}
141+
142+
if (options.storage_engine()) {
143+
options_builder << "storageEngine" << *options.storage_engine();
144+
}
145+
146146
if (options.validation_criteria()) {
147147
auto validation_level_to_string = [](validation_criteria::validation_level level) {
148148
switch (level) {

0 commit comments

Comments
 (0)