From bd00326e1e2f58f8f38e30bccdc99aa9743eefb4 Mon Sep 17 00:00:00 2001 From: rjy7 Date: Thu, 8 Jul 2010 22:04:31 +0000 Subject: [PATCH] Added a comment --- .../edit/forms/js/addAuthorsToInformationResource.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/productMods/edit/forms/js/addAuthorsToInformationResource.js b/productMods/edit/forms/js/addAuthorsToInformationResource.js index 6baf4d8f..f22cbe85 100644 --- a/productMods/edit/forms/js/addAuthorsToInformationResource.js +++ b/productMods/edit/forms/js/addAuthorsToInformationResource.js @@ -521,8 +521,11 @@ var addAuthorForm = { // authorName = authorLink.html(); if (status === 'success') { - // Both these cases can be replaced by calling a reorder. And we could - // do away with the position span altogether in that case. + // Both these cases can be replaced by calling a reorder: + // positions are not needed if we always eliminate gaps in rank + // (what about an error on the reorder call, though?) + // The reset of the rank hidden form field is done in the + // reorder callback. if (nextAuthorships.length) { // Reset the position value of each succeeding authorship nextAuthorships.each(function() {