continue work on it

astappiev · astappiev · commit f58f16d471cb · 2024-01-22T12:25:13.000+01:00
diff --git a/src/main/java/de/l3s/learnweb/resource/Annotation.java b/src/main/java/de/l3s/learnweb/resource/Annotation.java
@@ -11,12 +11,13 @@ public class Annotation implements Serializable {
     @Serial
     private static final long serialVersionUID = 6321296603254649454L;
 
+    private int annotationId;
     private int resourceId;
     private int userId;
     private String action;
     private String selection;
     private String annotation;
-    private Instant timestamp;
+    private Instant createdAt;
 
     // cached values
     private transient User user;
@@ -25,13 +26,12 @@ public class Annotation implements Serializable {
     public Annotation() {
     }
 
-    public Annotation(int resourceId, int userId, String action, String selection, String annotation) {
-        this.resourceId = resourceId;
-        this.userId = userId;
-        this.action = action;
-        this.selection = selection;
-        this.annotation = annotation;
-        this.timestamp = Instant.now();
+    public int getAnnotationId() {
+        return annotationId;
+    }
+
+    public void setAnnotationId(final int annotationId) {
+        this.annotationId = annotationId;
     }
 
     public int getResourceId() {
@@ -74,17 +74,17 @@ public void setAction(String action) {
         this.action = action;
     }
 
-    public Instant getTimestamp() {
-        return timestamp;
+    public Instant getCreatedAt() {
+        return createdAt;
     }
 
-    public void setTimestamp(Instant timestamp) {
-        this.timestamp = timestamp;
+    public void setCreatedAt(Instant createdAt) {
+        this.createdAt = createdAt;
     }
 
     public User getUser() {
         if (null == user) {
-            user = Learnweb.dao().getUserDao().findByIdOrElseThrow(getUserId());
+            user = Learnweb.dao().getUserDao().findByIdOrElseThrow(userId);
         }
         return user;
     }
diff --git a/src/main/java/de/l3s/learnweb/resource/AnnotationDao.java b/src/main/java/de/l3s/learnweb/resource/AnnotationDao.java
@@ -17,24 +17,28 @@
 
 import de.l3s.util.SqlHelper;
 
+@RegisterRowMapper(AnnotationDao.AnnotationMapper.class)
 public interface AnnotationDao extends SqlObject, Serializable {
 
-    @RegisterRowMapper(AnnotationMapper.class)
     @SqlQuery("SELECT a.* FROM lw_transcript_log a JOIN lw_resource USING(resource_id) WHERE user_id IN(<userIds>) and deleted = 0 ORDER BY user_id, created_at DESC")
     List<Annotation> findLogsByUserIds(@BindList("userIds") Collection<Integer> userIds);
 
+    @SqlQuery("SELECT a.* FROM lw_resource_annotation a WHERE resource_id = ?")
+    List<Annotation> findAllByResourceId(int resourceId);
+
     default void save(Annotation annotation) {
-        if (annotation.getTimestamp() == null) {
-            annotation.setTimestamp(Instant.now());
+        if (annotation.getCreatedAt() == null) {
+            annotation.setCreatedAt(Instant.now());
         }
 
         LinkedHashMap<String, Object> params = new LinkedHashMap<>();
+        params.put("annotation_id", annotation.getAnnotationId());
         params.put("resource_id", annotation.getResourceId());
         params.put("user_id", annotation.getUserId());
-        params.put("action", annotation.getAction());
+        params.put("action", SqlHelper.toNullable(annotation.getAction()));
         params.put("selection", SqlHelper.toNullable(annotation.getSelection()));
         params.put("annotation", SqlHelper.toNullable(annotation.getAnnotation()));
-        params.put("created_at", annotation.getTimestamp());
+        params.put("created_at", annotation.getCreatedAt());
 
         SqlHelper.handleSave(getHandle(), "lw_resource_annotation", params).execute();
     }
@@ -43,12 +47,13 @@ class AnnotationMapper implements RowMapper<Annotation> {
         @Override
         public Annotation map(final ResultSet rs, final StatementContext ctx) throws SQLException {
             Annotation log = new Annotation();
+            log.setAnnotationId(rs.getInt("annotation_id"));
             log.setResourceId(rs.getInt("resource_id"));
             log.setUserId(rs.getInt("user_id"));
             log.setAction(rs.getString("action"));
-            log.setSelection(rs.getString("words_selected"));
-            log.setAnnotation(rs.getString("user_annotation"));
-            log.setTimestamp(rs.getTimestamp("created_at").toInstant());
+            log.setSelection(rs.getString("selection"));
+            log.setAnnotation(rs.getString("annotation"));
+            log.setCreatedAt(rs.getTimestamp("created_at").toInstant());
             return log;
         }
     }
diff --git a/src/main/java/de/l3s/learnweb/resource/Resource.java b/src/main/java/de/l3s/learnweb/resource/Resource.java
@@ -541,9 +541,9 @@ public void setTypeFromFormat(String format) {
             return;
         }
 
-        if (format.equals("text/html") || format.equals("application/xhtml+xml")) {
+        if (StringUtils.equalsAny(format, "text/html", "application/xhtml+xml")) {
             this.type = ResourceType.website;
-        } else if (format.startsWith("text/") || StringUtils.equalsAny(format, "application/json", "application/xml")) {
+        } else if (format.startsWith("text/") || StringUtils.equalsAny(format, "application/json", "application/xml", "application/sql")) {
             this.type = ResourceType.text;
         } else if (format.startsWith("image/")) {
             this.type = ResourceType.image;
diff --git a/src/main/java/de/l3s/learnweb/resource/ResourceAnnotationBean.java b/src/main/java/de/l3s/learnweb/resource/ResourceAnnotationBean.java
@@ -1,8 +1,11 @@
 package de.l3s.learnweb.resource;
 
+import java.io.IOException;
 import java.io.Serial;
 import java.io.Serializable;
+import java.nio.file.Files;
 import java.util.ArrayList;
+import java.util.List;
 import java.util.Map;
 import java.util.regex.Pattern;
 
@@ -12,10 +15,15 @@
 import jakarta.inject.Inject;
 import jakarta.inject.Named;
 
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
+import org.omnifaces.util.Beans;
 import org.omnifaces.util.Faces;
 import org.omnifaces.util.Messages;
 import org.primefaces.PrimeFaces;
 
+import com.google.gson.Gson;
+
 import de.l3s.learnweb.beans.ApplicationBean;
 import de.l3s.learnweb.beans.BeanAssert;
 import de.l3s.util.NlpHelper;
@@ -25,15 +33,42 @@
 public class ResourceAnnotationBean extends ApplicationBean implements Serializable {
     @Serial
     private static final long serialVersionUID = -6100755972011969429L;
+    private static final Logger log = LogManager.getLogger(ResourceAnnotationBean.class);
 
     private static final Pattern SPACES = Pattern.compile("\\s+");
+    private Resource resource;
 
     @Inject
     private AnnotationDao annotationDao;
 
     @PostConstruct
     public void onLoad() {
         BeanAssert.authorized(isLoggedIn());
+
+        resource = Beans.getInstance(ResourceDetailBean.class).getResource();
+    }
+
+    public void convert() {
+        if (resource.getType() == ResourceType.text) {
+            if (resource.getMainFile() != null) {
+                try {
+                    String lines = Files.readString(resource.getMainFile().getActualFile().toPath());
+                    lines = lines.replace("\n", "<br/>");
+                    resource.setTranscript(lines);
+                    resource.save();
+                    return;
+                } catch (IOException e) {
+                    log.error("Error reading file while extracting transcript", e);
+                }
+            }
+        }
+
+        addGrowl(FacesMessage.SEVERITY_ERROR, "Unfortunately, we could not extract the text from the file.");
+    }
+
+    public String getAnnotationsAsJson() {
+        List<Annotation> annotations = annotationDao.findAllByResourceId(resource.getId());
+        return new Gson().toJson(annotations);
     }
 
     /**
@@ -46,8 +81,6 @@ public void commandSaveAnnotation(final Resource resource) {
         resource.setTranscript(annotatedText);
         resource.save();
 
-        annotationDao.save(new Annotation(resource.getId(), getUser().getId(), "save transcript", null, null));
-
         getUser().clearCaches();
         addGrowl(FacesMessage.SEVERITY_INFO, "Changes_saved");
     }
@@ -63,8 +96,6 @@ public void commandCommitAnnotation(final Resource resource) {
         resource.setReadOnlyTranscript(true);
         resource.save();
 
-        annotationDao.save(new Annotation(resource.getId(), getUser().getId(), "submit transcript", null, null));
-
         getUser().clearCaches();
         addGrowl(FacesMessage.SEVERITY_INFO, "Annotation Submitted");
     }
@@ -76,7 +107,13 @@ public void commandLogAnnotation(final Resource resource) {
         BeanAssert.authorized(resource.canAnnotateResource(getUser()));
 
         Map<String, String> params = Faces.getRequestParameterMap();
-        annotationDao.save(new Annotation(resource.getId(), getUser().getId(), params.get("action"), params.get("selection"), params.get("annotation")));
+        Annotation annotation = new Annotation();
+        annotation.setResourceId(resource.getId());
+        annotation.setUserId(getUser().getId());
+        annotation.setAction(params.get("action"));
+        annotation.setSelection(params.get("selection"));
+        annotation.setAnnotation(params.get("annotation"));
+        annotationDao.save(annotation);
     }
 
     /**
diff --git a/src/main/java/de/l3s/learnweb/resource/ResourceType.java b/src/main/java/de/l3s/learnweb/resource/ResourceType.java
@@ -14,9 +14,9 @@ public enum ResourceType {
     audio,
     pdf,
     website,
-    spreadsheet,
-    presentation,
-    document,
+    spreadsheet, // onlyoffice
+    presentation, // onlyoffice
+    document, // onlyoffice
     file, // applications, archives, etc
 
     // learnweb types
diff --git a/src/main/java/de/l3s/learnweb/resource/ted/TedTranscriptBean.java b/src/main/java/de/l3s/learnweb/resource/ted/TedTranscriptBean.java
@@ -2,10 +2,10 @@
 
 import java.io.Serial;
 import java.io.Serializable;
+import java.util.ArrayList;
 import java.util.LinkedList;
 import java.util.List;
 import java.util.Locale;
-import java.util.Map;
 
 import jakarta.faces.model.SelectItem;
 import jakarta.faces.view.ViewScoped;
@@ -120,7 +120,7 @@ public void setTranscript() {
         });
 
         // TODO: but why do we need this (two lines below)?
-        String transcript = sb.toString().replaceAll("\n", "<br/><br/>");
+        String transcript = sb.toString().replace("\n", "<br/><br/>");
         Document doc = Jsoup.parse(transcript);
         tedResource.setTranscript(doc.getElementsByTag("body").html());
     }
@@ -172,7 +172,7 @@ public List<SelectItem> getLanguageList() {
             if (langList.isEmpty()) {
                 languageList.add(new SelectItem("NA", "No Transcripts Available"));
             } else {
-                List<Locale> locales = new ArrayList<>();
+                ArrayList<Locale> locales = new ArrayList<>();
                 try {
                     for (String lang : langList) {
                         locales.add(LocaleUtils.toLocale(lang));
diff --git a/src/main/resources/db/migration/V14__Add_Resource_Annotation.sql b/src/main/resources/db/migration/V14__Add_Resource_Annotation.sql
@@ -1,8 +1,9 @@
 CREATE TABLE IF NOT EXISTS `lw_resource_annotation` (
+    `annotation_id` INT(10) UNSIGNED NOT NULL PRIMARY KEY AUTO_INCREMENT,
     `resource_id` INT(10) UNSIGNED NOT NULL,
     `user_id` INT(10) UNSIGNED NOT NULL,
     `action` CHAR(25) NOT NULL,
     `selection` LONGTEXT NOT NULL,
-    `annotation` MEDIUMTEXT NOT NULL,
+    `annotation` MEDIUMTEXT NULL,
     `created_at` TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP()
 );
diff --git a/src/main/webapp/WEB-INF/templates/blocks/resourceview/embedded_text.xhtml b/src/main/webapp/WEB-INF/templates/blocks/resourceview/embedded_text.xhtml
diff --git a/src/main/webapp/lw/admin/detailed_transcript_log.xhtml b/src/main/webapp/lw/admin/detailed_transcript_log.xhtml
diff --git a/src/main/webapp/lw/ted_transcript.xhtml b/src/main/webapp/lw/ted_transcript.xhtml
