From 9a9784c63232ac4f2c95eeeabd34cc1960959a19 Mon Sep 17 00:00:00 2001
From: nac26 <nac26@43c2fc9b-1942-4403-87b2-35b4218f7473>
Date: Wed, 14 Jul 2010 13:28:11 +0000
Subject: [PATCH] removed links on author names for author form and replaced
 with <span> NIHVIVO-646

---
 .../edit/forms/addAuthorsToInformationResource.jsp     | 10 +++++-----
 .../edit/forms/css/addAuthorsToInformationResource.css |  6 +++---
 .../edit/forms/js/addAuthorsToInformationResource.js   | 10 +++++-----
 3 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/productMods/edit/forms/addAuthorsToInformationResource.jsp b/productMods/edit/forms/addAuthorsToInformationResource.jsp
index 1205a277..e29e4b73 100644
--- a/productMods/edit/forms/addAuthorsToInformationResource.jsp
+++ b/productMods/edit/forms/addAuthorsToInformationResource.jsp
@@ -332,24 +332,24 @@ SPARQL queries for existing values. --%>
             <span class="position" id="${position}"></span> 
 
             <%-- span.author will be used in the next phase, when we display a message that the author has been
-            removed. That text will replace the a.authorLink, which will be removed. --%>    
+            removed. That text will replace the a.authorName, which will be removed. --%>    
             <span class="author">
-                <%-- This span is here to assign a width to. We can't assign directly to the a.authorLink,
+                <%-- This span is here to assign a width to. We can't assign directly to the a.authorName,
                 for the case when it's followed by an em tag - we want the width to apply to the whole thing. --%>
-                <span class="authorLinkWrapper">
+                <span class="authorNameWrapper">
 	                <c:choose>
 	                    <c:when test="${!empty author}">
 	                        <c:url var="authorHref" value="/individual">
 	                            <c:param name="uri" value="${author.URI}"/>
 	                        </c:url> 
-	                        <a href="#" id="${author.URI}" class="authorLink">${author.name}</a>
+	                        <span class="authorName">${author.name}</span>
 	                    </c:when>
 	
 	                    <c:otherwise>
 		                   <c:url var="authorshipHref" value="/individual">
 		                       <c:param name="uri" value="${authorshipUri}"/>
 		                   </c:url>                
-		                   <a href="${authorshipHref}" id="${authorshipUri}" class="authorLink">${authorshipName}</a><em> (no linked author)</em>
+		                   <a href="${authorshipHref}" id="${authorshipUri}" class="authorName">${authorshipName}</a><em> (no linked author)</em>
 	                    </c:otherwise>
 	                </c:choose>
                 </span>
diff --git a/productMods/edit/forms/css/addAuthorsToInformationResource.css b/productMods/edit/forms/css/addAuthorsToInformationResource.css
index 0fcc27d3..500f2677 100644
--- a/productMods/edit/forms/css/addAuthorsToInformationResource.css
+++ b/productMods/edit/forms/css/addAuthorsToInformationResource.css
@@ -15,14 +15,14 @@
     background: url("../images/sortable_icon.png") no-repeat left center;    
 }
 
-#authorships .authorLinkWrapper {
+#authorships .authorNameWrapper {
     display: inline-block;
     width: 15em;
 }
 
-#authorships.dd .authorLinkWrapper,
+#authorships.dd .authorNameWrapper,
 #authorships.dd li,
-#authorships.dd a.authorLink {
+#authorships.dd span.authorName {
     cursor: move;
 }
 
diff --git a/productMods/edit/forms/js/addAuthorsToInformationResource.js b/productMods/edit/forms/js/addAuthorsToInformationResource.js
index be41d774..aaf8e225 100644
--- a/productMods/edit/forms/js/addAuthorsToInformationResource.js
+++ b/productMods/edit/forms/js/addAuthorsToInformationResource.js
@@ -202,7 +202,7 @@ var addAuthorForm = {
 
     setAcFilter: function() {
 
-        var existingAuthors = $('#authorships .authorLink'); 
+        var existingAuthors = $('#authorships .authorName'); 
         this.acFilter = [];
         
         existingAuthors.each(function() {
@@ -272,7 +272,7 @@ var addAuthorForm = {
             return;
         }
         
-        $('.authorLinkWrapper').each(function() {
+        $('.authorNameWrapper').each(function() {
             $(this).attr('title', 'Drag and drop to reorder authors');
         });
         
@@ -534,7 +534,7 @@ var addAuthorForm = {
             complete: function(request, status) {
                 var authorship = $(this).parents('.authorship'),
                     nextAuthorships = authorship.nextAll(),
-                    author = authorship.find('.authorLink').attr('id'),
+                    author = authorship.find('.authorName').attr('id'),
                     rank;
 //                  author = $(this).siblings('span.author'),
 //                  authorLink = author.children('a.authorLink'),
@@ -591,7 +591,7 @@ var addAuthorForm = {
     disableAuthorDD: function() {
     	var authorships = $('#authorships'),
             authorship = $('.authorship'),
-            authorLinkWrapper = $('.authorLinkWrapper');
+            authorNameWrapper = $('.authorNameWrapper');
             
     	authorships.sortable({ disable: true } );
         authorships.removeClass('dd');
@@ -599,7 +599,7 @@ var addAuthorForm = {
     	authorship.css('background', 'none');
     	authorship.css('padding-left', '0');
         
-    	authorLinkWrapper.attr('title', '');
+    	authorNameWrapper.attr('title', '');
     },
 
     // RY To be implemented later.