read_ids: function (ids, fields, options) {
if (_.isEmpty(ids))
return $.Deferred().resolve([]);
-
+
options = options || {};
- // TODO: reorder results to match ids list
- return this._model.call('read',
- [ids, fields || false],
- {context: this.get_context(options.context)});
+ var method = 'read';
+ var ids_arg = ids;
+ var context = this.get_context(options.context);
+ if (options.check_access_rule === true){
+ method = 'search_read';
+ ids_arg = [['id', 'in', ids]];
+ context = new instance.web.CompoundContext(context, {active_test: false});
+ }
+ return this._model.call(method,
+ [ids_arg, fields || false],
+ {context: context})
+ .then(function (records) {
+ if (records.length <= 1) { return records; }
+ var indexes = {};
+ for (var i = 0; i < ids.length; i++) {
+ indexes[ids[i]] = i;
+ }
+ records.sort(function (a, b) {
+ return indexes[a.id] - indexes[b.id];
+ });
+ return records;
+ });
},
/**
* Read a slice of the records represented by this DataSet, based on its
} else {
var fields = _.keys(self.fields_view.fields);
fields.push('display_name');
- // Use of search_read instead of read to check if we can still read the record (security rules)
- return self.dataset.call('search_read', [[['id', '=', self.dataset.ids[self.dataset.index]]], fields],
+ return self.dataset.read_index(fields,
{
context: {
'bin_size': true,
'future_display_name': true
- }
+ },
+ check_access_rule: true
}).then(function(r) {
- if (_.isEmpty(r)){
- self.do_action('history_back');
- }
- else{
- self.trigger('load_record', r[0]);
- }
+ self.trigger('load_record', r);
+ }).fail(function (){
+ self.do_action('history_back');
});
}
});
reload_record: function (record) {
var self = this;
// Use of search_read instead of read to check if we can still read the record (security rules)
- return this.dataset.call('search_read', [
- [['id', '=', record.get('id')]],
+ return this.dataset.read_ids(
+ [record.get('id')],
_.pluck(_(this.columns).filter(function (r) {
return r.tag === 'field';
- }), 'name')]
+ }), 'name'),
+ {check_access_rule: true}
).done(function (records) {
- var values = _.isEmpty(records) ? undefined : records[0];
+ var values = records[0];
if (!values) {
self.records.remove(record);
return;