Bring back delete permission in share overview
This commit is contained in:
parent
73d9699be9
commit
dcfaf3afdc
|
@ -105,11 +105,7 @@
|
||||||
},
|
},
|
||||||
|
|
||||||
getDirectoryPermissions: function() {
|
getDirectoryPermissions: function() {
|
||||||
var perms = OC.PERMISSION_READ;
|
return OC.PERMISSION_READ | OC.PERMISSION_DELETE;
|
||||||
if (this._sharedWithUser) {
|
|
||||||
perms |= OC.PERMISSION_DELETE;
|
|
||||||
}
|
|
||||||
return perms;
|
|
||||||
},
|
},
|
||||||
|
|
||||||
updateStorageStatistics: function() {
|
updateStorageStatistics: function() {
|
||||||
|
@ -271,11 +267,7 @@
|
||||||
}
|
}
|
||||||
file.name = OC.basename(share.path);
|
file.name = OC.basename(share.path);
|
||||||
file.path = OC.dirname(share.path);
|
file.path = OC.dirname(share.path);
|
||||||
if (this._sharedWithUser) {
|
file.permissions = OC.PERMISSION_ALL;
|
||||||
file.permissions = OC.PERMISSION_ALL;
|
|
||||||
} else {
|
|
||||||
file.permissions = OC.PERMISSION_ALL - OC.PERMISSION_DELETE;
|
|
||||||
}
|
|
||||||
if (file.path) {
|
if (file.path) {
|
||||||
file.extraData = share.path;
|
file.extraData = share.path;
|
||||||
}
|
}
|
||||||
|
|
|
@ -85,7 +85,7 @@ describe('OCA.Sharing.FileList tests', function() {
|
||||||
file_source: 49,
|
file_source: 49,
|
||||||
file_target: '/local path/local name.txt',
|
file_target: '/local path/local name.txt',
|
||||||
path: 'files/something shared.txt',
|
path: 'files/something shared.txt',
|
||||||
permissions: OC.PERMISSION_ALL,
|
permissions: 31,
|
||||||
stime: 11111,
|
stime: 11111,
|
||||||
share_type: OC.Share.SHARE_TYPE_USER,
|
share_type: OC.Share.SHARE_TYPE_USER,
|
||||||
share_with: 'user1',
|
share_with: 'user1',
|
||||||
|
@ -159,8 +159,7 @@ describe('OCA.Sharing.FileList tests', function() {
|
||||||
expect($tr.attr('data-file')).toEqual('local name.txt');
|
expect($tr.attr('data-file')).toEqual('local name.txt');
|
||||||
expect($tr.attr('data-path')).toEqual('/local path');
|
expect($tr.attr('data-path')).toEqual('/local path');
|
||||||
expect($tr.attr('data-size')).not.toBeDefined();
|
expect($tr.attr('data-size')).not.toBeDefined();
|
||||||
expect(parseInt($tr.attr('data-permissions'), 10))
|
expect($tr.attr('data-permissions')).toEqual('31'); // read and delete
|
||||||
.toEqual(OC.PERMISSION_ALL); // read and delete
|
|
||||||
expect($tr.attr('data-mime')).toEqual('text/plain');
|
expect($tr.attr('data-mime')).toEqual('text/plain');
|
||||||
expect($tr.attr('data-mtime')).toEqual('11111000');
|
expect($tr.attr('data-mtime')).toEqual('11111000');
|
||||||
expect($tr.attr('data-share-owner')).toEqual('User Two');
|
expect($tr.attr('data-share-owner')).toEqual('User Two');
|
||||||
|
@ -236,8 +235,7 @@ describe('OCA.Sharing.FileList tests', function() {
|
||||||
expect($tr.attr('data-file')).toEqual('local name');
|
expect($tr.attr('data-file')).toEqual('local name');
|
||||||
expect($tr.attr('data-path')).toEqual('/local path');
|
expect($tr.attr('data-path')).toEqual('/local path');
|
||||||
expect($tr.attr('data-size')).not.toBeDefined();
|
expect($tr.attr('data-size')).not.toBeDefined();
|
||||||
expect(parseInt($tr.attr('data-permissions'), 10))
|
expect($tr.attr('data-permissions')).toEqual('31'); // read and delete
|
||||||
.toEqual(OC.PERMISSION_ALL); // read and delete
|
|
||||||
expect($tr.attr('data-mime')).toEqual('httpd/unix-directory');
|
expect($tr.attr('data-mime')).toEqual('httpd/unix-directory');
|
||||||
expect($tr.attr('data-mtime')).toEqual('11111000');
|
expect($tr.attr('data-mtime')).toEqual('11111000');
|
||||||
expect($tr.attr('data-share-owner')).toEqual('User Two');
|
expect($tr.attr('data-share-owner')).toEqual('User Two');
|
||||||
|
@ -333,8 +331,7 @@ describe('OCA.Sharing.FileList tests', function() {
|
||||||
expect($tr.attr('data-file')).toEqual('local name.txt');
|
expect($tr.attr('data-file')).toEqual('local name.txt');
|
||||||
expect($tr.attr('data-path')).toEqual('/local path');
|
expect($tr.attr('data-path')).toEqual('/local path');
|
||||||
expect($tr.attr('data-size')).not.toBeDefined();
|
expect($tr.attr('data-size')).not.toBeDefined();
|
||||||
expect(parseInt($tr.attr('data-permissions'), 10))
|
expect($tr.attr('data-permissions')).toEqual('31'); // read and delete
|
||||||
.toEqual(OC.PERMISSION_ALL - OC.PERMISSION_DELETE); // read
|
|
||||||
expect($tr.attr('data-mime')).toEqual('text/plain');
|
expect($tr.attr('data-mime')).toEqual('text/plain');
|
||||||
expect($tr.attr('data-mtime')).toEqual('11111000');
|
expect($tr.attr('data-mtime')).toEqual('11111000');
|
||||||
expect($tr.attr('data-share-owner')).not.toBeDefined();
|
expect($tr.attr('data-share-owner')).not.toBeDefined();
|
||||||
|
@ -375,8 +372,7 @@ describe('OCA.Sharing.FileList tests', function() {
|
||||||
expect($tr.attr('data-file')).toEqual('local name');
|
expect($tr.attr('data-file')).toEqual('local name');
|
||||||
expect($tr.attr('data-path')).toEqual('/local path');
|
expect($tr.attr('data-path')).toEqual('/local path');
|
||||||
expect($tr.attr('data-size')).not.toBeDefined();
|
expect($tr.attr('data-size')).not.toBeDefined();
|
||||||
expect(parseInt($tr.attr('data-permissions'), 10))
|
expect($tr.attr('data-permissions')).toEqual('31'); // read and delete
|
||||||
.toEqual(OC.PERMISSION_ALL - OC.PERMISSION_DELETE); // read
|
|
||||||
expect($tr.attr('data-mime')).toEqual('httpd/unix-directory');
|
expect($tr.attr('data-mime')).toEqual('httpd/unix-directory');
|
||||||
expect($tr.attr('data-mtime')).toEqual('11111000');
|
expect($tr.attr('data-mtime')).toEqual('11111000');
|
||||||
expect($tr.attr('data-share-owner')).not.toBeDefined();
|
expect($tr.attr('data-share-owner')).not.toBeDefined();
|
||||||
|
@ -427,8 +423,7 @@ describe('OCA.Sharing.FileList tests', function() {
|
||||||
expect($tr.attr('data-file')).toEqual('local name.txt');
|
expect($tr.attr('data-file')).toEqual('local name.txt');
|
||||||
expect($tr.attr('data-path')).toEqual('/local path');
|
expect($tr.attr('data-path')).toEqual('/local path');
|
||||||
expect($tr.attr('data-size')).not.toBeDefined();
|
expect($tr.attr('data-size')).not.toBeDefined();
|
||||||
expect(parseInt($tr.attr('data-permissions'), 10))
|
expect($tr.attr('data-permissions')).toEqual('31'); // read and delete
|
||||||
.toEqual(OC.PERMISSION_ALL - OC.PERMISSION_DELETE); // read
|
|
||||||
expect($tr.attr('data-mime')).toEqual('text/plain');
|
expect($tr.attr('data-mime')).toEqual('text/plain');
|
||||||
expect($tr.attr('data-mtime')).toEqual('11111000');
|
expect($tr.attr('data-mtime')).toEqual('11111000');
|
||||||
expect($tr.attr('data-share-owner')).not.toBeDefined();
|
expect($tr.attr('data-share-owner')).not.toBeDefined();
|
||||||
|
@ -496,8 +491,7 @@ describe('OCA.Sharing.FileList tests', function() {
|
||||||
expect($tr.attr('data-file')).toEqual('local name.txt');
|
expect($tr.attr('data-file')).toEqual('local name.txt');
|
||||||
expect($tr.attr('data-path')).toEqual('/local path');
|
expect($tr.attr('data-path')).toEqual('/local path');
|
||||||
expect($tr.attr('data-size')).not.toBeDefined();
|
expect($tr.attr('data-size')).not.toBeDefined();
|
||||||
expect(parseInt($tr.attr('data-permissions'), 10))
|
expect($tr.attr('data-permissions')).toEqual('31'); // read and delete
|
||||||
.toEqual(OC.PERMISSION_ALL - OC.PERMISSION_DELETE); // read
|
|
||||||
expect($tr.attr('data-mime')).toEqual('text/plain');
|
expect($tr.attr('data-mime')).toEqual('text/plain');
|
||||||
// always use the most recent stime
|
// always use the most recent stime
|
||||||
expect($tr.attr('data-mtime')).toEqual('22222000');
|
expect($tr.attr('data-mtime')).toEqual('22222000');
|
||||||
|
@ -591,8 +585,7 @@ describe('OCA.Sharing.FileList tests', function() {
|
||||||
expect($tr.attr('data-file')).toEqual('local name.txt');
|
expect($tr.attr('data-file')).toEqual('local name.txt');
|
||||||
expect($tr.attr('data-path')).toEqual('/local path');
|
expect($tr.attr('data-path')).toEqual('/local path');
|
||||||
expect($tr.attr('data-size')).not.toBeDefined();
|
expect($tr.attr('data-size')).not.toBeDefined();
|
||||||
expect(parseInt($tr.attr('data-permissions'), 10))
|
expect($tr.attr('data-permissions')).toEqual('31'); // read and delete
|
||||||
.toEqual(OC.PERMISSION_ALL - OC.PERMISSION_DELETE); // read
|
|
||||||
expect($tr.attr('data-mime')).toEqual('text/plain');
|
expect($tr.attr('data-mime')).toEqual('text/plain');
|
||||||
expect($tr.attr('data-mtime')).toEqual('11111000');
|
expect($tr.attr('data-mtime')).toEqual('11111000');
|
||||||
expect($tr.attr('data-share-recipients')).not.toBeDefined();
|
expect($tr.attr('data-share-recipients')).not.toBeDefined();
|
||||||
|
@ -633,8 +626,7 @@ describe('OCA.Sharing.FileList tests', function() {
|
||||||
expect($tr.attr('data-file')).toEqual('local name.txt');
|
expect($tr.attr('data-file')).toEqual('local name.txt');
|
||||||
expect($tr.attr('data-path')).toEqual('/local path');
|
expect($tr.attr('data-path')).toEqual('/local path');
|
||||||
expect($tr.attr('data-size')).not.toBeDefined();
|
expect($tr.attr('data-size')).not.toBeDefined();
|
||||||
expect(parseInt($tr.attr('data-permissions'), 10))
|
expect($tr.attr('data-permissions')).toEqual('31'); // read and delete
|
||||||
.toEqual(OC.PERMISSION_ALL - OC.PERMISSION_DELETE); // read
|
|
||||||
expect($tr.attr('data-mime')).toEqual('text/plain');
|
expect($tr.attr('data-mime')).toEqual('text/plain');
|
||||||
expect($tr.attr('data-mtime')).toEqual('11111000');
|
expect($tr.attr('data-mtime')).toEqual('11111000');
|
||||||
expect($tr.attr('data-share-recipients')).not.toBeDefined();
|
expect($tr.attr('data-share-recipients')).not.toBeDefined();
|
||||||
|
|
Loading…
Reference in New Issue