Keep error message after generateCodemeta() call

Signed-off-by: Arthit Suriyawongkul <arthit@gmail.com>

Author: bact

js/codemeta_generation.js

@@ -429,12 +429,17 @@ async function recompactDocWithAllContexts(doc) {
 async function importCodemeta() {
     var inputForm = document.querySelector('#inputForm');
     var doc = parseAndValidateCodemeta(false);
+    // parseAndValidateCodemeta() may set an error message.
+    // resetForm() below runs generateCodemeta() which would clear the error message;
+    // preserve the error message so it is there after import.
+    const validationError = (document.getElementById('errorMessage') || {}).textContent || '';
 
     // Re-compact document with all contexts
     // to allow importing property from any context
     doc = await recompactDocWithAllContexts(doc);
 
     resetForm();
+    if (validationError) setError(validationError);
 
     if (doc['license'] !== undefined) {
         if (typeof doc['license'] === 'string') {

js/dynamic_form.js

@@ -232,10 +232,8 @@ function removePersons(prefix) {
     const container = document.getElementById(`${prefix}_list`);
     if (!container) return;
 
-    const persons = Array.from(container.querySelectorAll('.person'));
-    persons.forEach(p => p.remove());
-
-    renumberPersons(prefix);
+    container.innerHTML = '';
+    setNbPersons(prefix, 0);
     generateCodemeta();
 }
 
@@ -273,6 +271,7 @@ function addRole(personPrefix) {
 
 function removeRole(personPrefix, roleIndex) {
     document.getElementById(`${personPrefix}_role_${roleIndex}`).remove();
+    generateCodemeta();
 }
 
 function resetForm() {