allow new terms to be added after populating a CV from OLS #1653

seek4science · Nov 10, 2023 · 769acca · 769acca
1 parent f6bdffd
commit 769acca
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 24 deletions.
diff --git a/app/assets/javascripts/controlled_vocabs.js.erb b/app/assets/javascripts/controlled_vocabs.js.erb
@@ -135,23 +135,6 @@ CVTerms = {
             }
         });
     },
-    disableAddTermButton: function(reason) {
-        $j('a#add-term').attr('disabled', true);
-        if (!$j('a#add-term').parent().is('span')) {
-            let span = $j('a#add-term').wrap('<span />').parent();
-            $j(span).attr('data-tooltip', reason);
-            bindTooltips('div#controlled-vocab-terms');
-            $j(span).click(function () {
-                alert(reason);
-            });
-        }
-    },
-    enableAddTermButton: function() {
-        let button = $j('a#add-term');
-        button.attr('disabled',false);
-        button.removeClass('disabled');
-        button.unwrap('span');
-    },
     // sets up and trigger the ajax call to fetch the terms list
     fetchTermsAJAX: function () {
         $j('div#controlled-vocab-terms').hide();
@@ -209,10 +192,8 @@ CVTerms = {
             if (selected.value == "") {
                 $j('#sample_controlled_vocab_ols_root_term_uri').val('');
                 $j('#ontology-root-uri').hide();
-                CVTerms.enableAddTermButton();
             } else {
                 $j('#ontology-root-uri').show();
-                CVTerms.disableAddTermButton('<%= I18n.t('controlled_vocabs.add_new_term_disabled_reason') %>');
                 var link = $j('a#selected-ols-link');
                 link.text(selected.text);
                 link.attr('href', '<%= Ebi::OlsClient::ROOT_URL %>/ontologies/' + selected.value);

diff --git a/app/views/sample_controlled_vocabs/_form.html.erb b/app/views/sample_controlled_vocabs/_form.html.erb
@@ -103,11 +103,7 @@
         </table>
       </div>
       <div>
-        <%
-          options = {id: 'add-term'}
-          options.merge!({disabled:true, disabled_reason:t('controlled_vocabs.add_new_term_disabled_reason')}) if @sample_controlled_vocab.ontology_based?
-        %>
-        <%= button_link_to('Add new term', 'add', '#', options) %>
+        <%= button_link_to('Add new term', 'add', '#', id: 'add-term') %>
         <%= button_link_to('Remove all terms', 'destroy', '#', :id => 'clear-terms', class:'btn btn-danger') %>
       </div>
     </div>