From dc1c6c282f5ce2dd2cefbd30fcd357dfc8ffb9ea Mon Sep 17 00:00:00 2001
From: Stanislaw Adaszewski <s.adaszewski@gmail.com>
Date: Wed, 5 Feb 2020 18:13:50 +0100
Subject: [PATCH] Changed default collection url to content browser, added
 properties as optional action in the listing.

---
 frontend/src/js/component/wb-collection-listing.js | 13 +++++++++++--
 frontend/src/js/component/wb-name-and-uuid.js      |  2 +-
 frontend/src/js/misc/url-for-object.js             | 11 +++++++----
 3 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/frontend/src/js/component/wb-collection-listing.js b/frontend/src/js/component/wb-collection-listing.js
index 993839c..90caaf1 100644
--- a/frontend/src/js/component/wb-collection-listing.js
+++ b/frontend/src/js/component/wb-collection-listing.js
@@ -39,7 +39,16 @@ class WBCollectionListing extends Component {
         <div>{ item.owner_uuid }</div>
       </div>),
       item['file_count'],
-      filesize(item['file_size_total'])
+      filesize(item['file_size_total']),
+      (<div>
+        <a class="btn btn-outline-primary m-1" title="Properties"
+          href={ urlForObject(item, 'properties') }>
+          <i class="fas fa-list-ul"></i>
+        </a>
+        <button class="btn btn-outline-danger m-1" title="Delete">
+          <i class="fas fa-trash"></i>
+        </button>
+      </div>)
     ]);
   }
 
@@ -109,7 +118,7 @@ class WBCollectionListing extends Component {
   render({ app, ownerUuid, activePage, getPageUrl }, { rows, numPages }) {
     return (
       <div>
-        <WBTable columns={ [ 'Name', 'Description', 'Owner', 'File Count', 'Total Size' ] }
+        <WBTable columns={ [ 'Name', 'Description', 'Owner', 'File Count', 'Total Size', 'Actions' ] }
           rows={ rows } />
 
         <WBPagination numPages={ numPages }
diff --git a/frontend/src/js/component/wb-name-and-uuid.js b/frontend/src/js/component/wb-name-and-uuid.js
index 9f8aefc..4f1eaf8 100644
--- a/frontend/src/js/component/wb-name-and-uuid.js
+++ b/frontend/src/js/component/wb-name-and-uuid.js
@@ -11,7 +11,7 @@ class WBNameAndUuid extends Component {
     if (!uuid)
       return;
 
-    if (uuid in lookup) {
+    if (lookup && (uuid in lookup)) {
       this.setState({ 'item': lookup[uuid]});
       return;
     }
diff --git a/frontend/src/js/misc/url-for-object.js b/frontend/src/js/misc/url-for-object.js
index eaba75c..31c17e1 100644
--- a/frontend/src/js/misc/url-for-object.js
+++ b/frontend/src/js/misc/url-for-object.js
@@ -1,6 +1,6 @@
 import arvadosTypeName from 'arvados-type-name';
 
-function urlForObject(item) {
+function urlForObject(item, mode='primary') {
   let objectType = arvadosTypeName(item.uuid.split('-')[1]);
   if (objectType === 'user')
     return ('/browse/' + item.uuid);
@@ -10,9 +10,12 @@ function urlForObject(item) {
     return ('/process/' + item.uuid);
   else if (objectType === 'workflow')
     return ('https://wb.arkau.roche.com/workflows/' + item.uuid);
-  else if (objectType === 'collection')
-    return ('/collection/' + item.uuid);
-  else if (objectType === 'container')
+  else if (objectType === 'collection') {
+    if (mode === 'primary' || mode === 'browse')
+      return ('/collection-browse/' + item.uuid);
+    else
+      return ('/collection/' + item.uuid);
+  } else if (objectType === 'container')
     return ('https://wb.arkau.roche.com/containers/' + item.uuid);
 }