Make fields on `InsertOption`, `UpdateOption`, etc public about arangors HOT 3 CLOSED

It is actually possible. The point is that TypedBuilder returns a new builder with a different type.

let builder = InsertOptions::builder();
let option = if some_condition {
   builder.wait_for_sync(true).build()
}
else{
   builder.build()
};

The pros of TypedBuilder is that we can know if the code is wrong at compile time and achieve zero cost abstraction. But I do think it a little cubersome in this case that you have to use a single match for all the conditions you want.

We can also create a builder for each condition:

let option = if some_condition {
   InsertOptions::builder().wait_for_sync(true).build()
}
else{
   InsertOptions::builder().build()
};

from arangors.

It is actually possible. The point is that TypedBuilder returns a new builder with a different type.

let builder = InsertOptions::builder();
let option = if some_condition {
   builder.wait_for_sync(true).build()
}
else{
   builder.build()
};

I tried this and it doesn't work. The second option works but I put a lof of options and My code looks like this:

impl Into<UpdateOptions> for OperationOptions {
    fn into(self) -> UpdateOptions {
        // TODO: remove this builder pattern whe arangors resolves https://github.com/fMeow/arangors/issues/71
        if let Some(value) = self.wait_for_sync {
            UpdateOptions::builder()
                .keep_null(true)
                .wait_for_sync(value)
                .ignore_revs(self.ignore_revs)
                .return_new(true) 
                .return_old(false)
                .silent(false)
                .build()
        } else {
            UpdateOptions::builder()
                .keep_null(true)
                .ignore_revs(self.ignore_revs)
                .return_new(true)
                .return_old(false)
                .silent(false)
                .build()
        }
    }
}

(source: https://gitlab.com/qonfucius/aragog under src/db/operation_options.rs)

from arangors.

I tried again and it works

It is actually possible. The point is that TypedBuilder returns a new builder with a different type.

let builder = InsertOptions::builder();
let option = if some_condition {
   builder.wait_for_sync(true).build()
}
else{
   builder.build()
};

The pros of TypedBuilder is that we can know if the code is wrong at compile time and achieve zero cost abstraction. But I do think it a little cubersome in this case that you have to use a single match for all the conditions you want.

We can also create a builder for each condition:

let option = if some_condition {
   InsertOptions::builder().wait_for_sync(true).build()
}
else{
   InsertOptions::builder().build()
};

from arangors.