Features about QuickInfo component:

* #13827, adding copy/paste a news functionality to quick info component.
* #13828, handling keywords at news level.

Author: SilverYoCha

quickinfo/quickinfo-configuration/src/main/config/properties/org/silverpeas/quickinfo/multilang/quickinfo.properties

@@ -38,6 +38,7 @@ quickinfo.news.broadcast.mode = Diffusion
 quickinfo.news.broadcast.mode.major = Importante
 quickinfo.news.broadcast.mode.ticker = D\u00e9filante
 quickinfo.news.broadcast.mode.blocking = Bloquante
+quickinfo.news.keywords = Mots cl\u00e9s
 
 quickinfo.news.untitled = Nouvelle actualit\u00e9
 quickinfo.news.draft.info = Cette actualit\u00e9 est actuellement un brouillon... <a href="#" onclick="javascript:publish()">Publiez-la maintenant !?</a>

quickinfo/quickinfo-configuration/src/main/config/properties/org/silverpeas/quickinfo/multilang/quickinfo_de.properties

@@ -38,6 +38,7 @@ quickinfo.news.broadcast.mode = Broadcast
 quickinfo.news.broadcast.mode.major = Important
 quickinfo.news.broadcast.mode.ticker = Ticker
 quickinfo.news.broadcast.mode.blocking = Blocking
+quickinfo.news.keywords=Schl\u00fcsselworte
 
 quickinfo.news.untitled = Untitled news
 quickinfo.news.draft.info = This news is a draft... <a href="#" onclick="javascript:publish()">Publish it now !?</a>

quickinfo/quickinfo-configuration/src/main/config/properties/org/silverpeas/quickinfo/multilang/quickinfo_en.properties

@@ -38,6 +38,7 @@ quickinfo.news.broadcast.mode = Broadcast
 quickinfo.news.broadcast.mode.major = Important
 quickinfo.news.broadcast.mode.ticker = Ticker
 quickinfo.news.broadcast.mode.blocking = Blocking
+quickinfo.news.keywords=Keywords
 
 quickinfo.news.untitled = Untitled news
 quickinfo.news.draft.info = This news is a draft... <a href="#" onclick="javascript:publish()">Publish it now !?</a>

quickinfo/quickinfo-configuration/src/main/config/properties/org/silverpeas/quickinfo/multilang/quickinfo_fr.properties

@@ -38,6 +38,7 @@ quickinfo.news.broadcast.mode = Diffusion
 quickinfo.news.broadcast.mode.major = Importante
 quickinfo.news.broadcast.mode.ticker = D\u00e9filante
 quickinfo.news.broadcast.mode.blocking = Bloquante
+quickinfo.news.keywords = Mots cl\u00e9s
 
 quickinfo.news.untitled = Nouvelle actualit\u00e9
 quickinfo.news.draft.info = Cette actualit\u00e9 est actuellement un brouillon... <a href="#" onclick="javascript:publish()">Publiez-la maintenant !?</a>

quickinfo/quickinfo-library/src/integration-test/java/org/silverpeas/components/quickinfo/repository/NewsRepositoryIT.java

@@ -87,8 +87,13 @@ public void testSaving() {
     OperationContext.fromUser(user);
     News savedNews = Transaction.performInOne(() -> {
       Period visibility = Period.between(LocalDate.now(), LocalDate.now().plusMonths(1));
-      News news =
-          new News("test", "test", visibility, true, true, false);
+      News news = News.builder()
+          .setTitleAndDescription("test", "test")
+          .setVisibilityPeriod(visibility)
+          .setImportant(true)
+          .setTicker(true)
+          .setMandatory(false)
+          .build();
       news.setComponentInstanceId("quickinfo1");
       news.setPublicationId("45789");
       news.createdBy(user);

quickinfo/quickinfo-library/src/main/java/org/silverpeas/components/quickinfo/model/DefaultQuickInfoService.java

@@ -35,11 +35,13 @@
 import org.silverpeas.components.quickinfo.service.QuickInfoContentManager;
 import org.silverpeas.components.quickinfo.service.QuickInfoDateComparatorDesc;
 import org.silverpeas.core.ResourceReference;
+import org.silverpeas.core.admin.component.model.PasteDetail;
 import org.silverpeas.core.admin.service.OrganizationController;
 import org.silverpeas.core.admin.service.OrganizationControllerProvider;
 import org.silverpeas.core.annotation.Service;
 import org.silverpeas.core.contribution.attachment.AttachmentServiceProvider;
 import org.silverpeas.core.contribution.attachment.model.Attachments;
+import org.silverpeas.core.contribution.attachment.model.DocumentType;
 import org.silverpeas.core.contribution.attachment.model.SimpleDocumentPK;
 import org.silverpeas.core.contribution.content.wysiwyg.service.WysiwygController;
 import org.silverpeas.core.contribution.contentcontainer.content.ContentManagerException;
@@ -49,6 +51,7 @@
 import org.silverpeas.core.contribution.publication.service.PublicationService;
 import org.silverpeas.core.i18n.I18NHelper;
 import org.silverpeas.core.index.indexing.model.IndexManager;
+import org.silverpeas.core.io.media.image.thumbnail.control.ThumbnailController;
 import org.silverpeas.core.io.upload.UploadedFile;
 import org.silverpeas.core.notification.system.ResourceEvent;
 import org.silverpeas.core.notification.user.client.constant.NotifAction;
@@ -89,6 +92,7 @@
 import static java.util.Collections.singletonList;
 import static java.util.function.Predicate.not;
 import static org.silverpeas.components.quickinfo.notification.QuickInfoDelayedVisibilityUserNotificationReminder.QUICKINFO_DELAYED_VISIBILITY_USER_NOTIFICATION;
+import static org.silverpeas.core.contribution.attachment.AttachmentServiceProvider.getAttachmentService;
 import static org.silverpeas.core.pdc.pdc.model.PdcClassification.aPdcClassificationOfContent;
 
 @Service
@@ -175,7 +179,7 @@ public boolean isRelatedTo(final String instanceId) {
   @Override
   @Transactional
   public News create(final News news) {
-    ResourceReference volatileAttachmentSourcePK =
+    ResourceReference volatileAttachmentSourceRef =
         new ResourceReference(news.getPublicationId(), news.getComponentInstanceId());
 
     // Creating publication
@@ -192,12 +196,12 @@ public News create(final News news) {
 
     // Attaching all documents linked to volatile news to the persisted news
     List<SimpleDocumentPK> movedDocumentPks = AttachmentServiceProvider.getAttachmentService()
-        .moveAllDocuments(volatileAttachmentSourcePK,
+        .moveAllDocuments(volatileAttachmentSourceRef,
             savedNews.getPublication().getPK().toResourceReference());
     if (!movedDocumentPks.isEmpty()) {
       // Change images path in wysiwyg
       WysiwygController.wysiwygPlaceHaveChanged(news.getComponentInstanceId(),
-          volatileAttachmentSourcePK.getId(), news.getComponentInstanceId(), savedNews.getId());
+          volatileAttachmentSourceRef.getId(), news.getComponentInstanceId(), savedNews.getId());
     }
 
     // Referring new content into taxonomy
@@ -213,6 +217,60 @@ public News create(final News news) {
     return savedNews;
   }
 
+  @Override
+  @Transactional
+  public News copyNews(final News newsToCopy, final PasteDetail pasteDetail) {
+
+    // Initializing the news instance
+    final News news = News.builder(newsToCopy).build();
+    news.setDraft();
+    news.setComponentInstanceId(pasteDetail.getToComponentId());
+    news.setCreatorId(pasteDetail.getUserId());
+
+    // Creating linked publication
+    PublicationDetail publication = news.getPublication();
+    publication.setIndexOperation(IndexManager.NONE);
+    final PublicationPK pubPK = getPublicationService().createPublication(publication);
+    publication = getPublicationService().getDetail(pubPK);
+
+    // Saving the new news instance into repository
+    news.setPublicationId(pubPK.getId());
+    news.setPublication(publication);
+    final News savedNews = newsRepository.save(news);
+
+    // Copying decorators
+    final ResourceReference pubSourceRef = newsToCopy.getPublication().getPK().toReference();
+    final ResourceReference pubDestRef = news.getPublication().getPK().toReference();
+    // - PDC
+    copyPdcPositions(newsToCopy, savedNews);
+    // - attachment files
+    getAttachmentService()
+        .listDocumentsByForeignKeyAndType(pubSourceRef, DocumentType.attachment, null)
+        .forEach(a -> getAttachmentService().copyDocument(a, pubDestRef));
+    // - WYSIWYG content
+    WysiwygController.copy(pubSourceRef.getComponentInstanceId(), pubSourceRef.getLocalId(),
+        pubDestRef.getComponentInstanceId(), pubDestRef.getLocalId(), pasteDetail.getUserId());
+    // - thumbnail
+    ThumbnailController.copyThumbnail(pubSourceRef, pubDestRef);
+
+    return savedNews;
+  }
+
+  private void copyPdcPositions(final News sourceNews, final News destNews) {
+    String sourceCmpId = sourceNews.getComponentInstanceId();
+    String destCmpId = destNews.getComponentInstanceId();
+    int sourceId = quickInfoContentManager.getOrCreateSilverContentId(sourceNews.getPublication());
+    int destId = quickInfoContentManager.getOrCreateSilverContentId(destNews.getPublication());
+    try {
+      pdcManager.copyPositions(sourceId, sourceCmpId, destId, destCmpId);
+    } catch (PdcException e) {
+      SilverLogger.getLogger(this).error(
+          "can not copy pdc positions from publication {0} of news {1} to publication {2} of news {3}",
+          new Object[]{sourceNews.getPublication().getPK(), sourceNews.getPK(),
+              destNews.getPublication().getPK(), destNews.getPK()}, e);
+    }
+  }
+
   @Override
   @Transactional
   public void publish(String id, String userId) {

quickinfo/quickinfo-library/src/main/java/org/silverpeas/components/quickinfo/model/News.java

@@ -122,19 +122,32 @@ public class News extends SilverpeasJpaEntity<News, UuidIdentifier> implements S
   @Column
   private String publishedBy;
 
-  protected News() {
-    // default constructor for the persistence engine
+  /**
+   * Gets a builder of {@link News} instances.
+   * @return a {@link Builder} instance
+   */
+  public static Builder builder() {
+    return new Builder();
   }
 
-  public News(String name, String description, Period visibilityPeriod, boolean important,
-      boolean ticker, boolean mandatory) {
-    this.publication = PublicationDetail.builder()
-        .setNameAndDescription(name, description)
-        .build();
-    this.publication.setVisibilityPeriod(visibilityPeriod);
-    setImportant(important);
-    setTicker(ticker);
-    setMandatory(mandatory);
+  /**
+   * Gets a builder of {@link News} instances from data of another one.
+   * @param news the news data to initialize with.
+   * @return a {@link Builder} instance
+   */
+  public static Builder builder(final News news) {
+    return builder()
+        .setTitleAndDescription(news.getTitle(), news.getDescription())
+        .setKeywords(news.getKeywords())
+        .setVisibilityPeriod(news.getVisibility().getPeriod())
+        .setImportant(news.isImportant())
+        .setTicker(news.isTicker())
+        .setMandatory(news.isMandatory());
+  }
+
+  protected News() {
+    // default constructor for the persistence engine
+    super();
   }
 
   public NewsPK getPK() {
@@ -168,6 +181,14 @@ public String getDescription() {
     return getPublication().getDescription();
   }
 
+  public void setKeywords(String keywords) {
+    getPublication().setKeywords(keywords);
+  }
+
+  public String getKeywords() {
+    return getPublication().getKeywords();
+  }
+
   public void setDescription(String desc) {
     getPublication().setDescription(desc);
   }
@@ -435,4 +456,87 @@ public boolean equals(final Object obj) {
   public ContributionModel getModel() {
     return new NewsModel(this);
   }
+
+  /**
+   * A builder of a {@link News} instance by setting some of its properties.
+   */
+  public static class Builder {
+
+    private final News news = new News();
+
+    public Builder() {
+      news.publication = PublicationDetail.builder().build();
+    }
+
+    /**
+     * Builds a {@link News} instance from the properties that were previously set with this 
+     * builder.
+     * @return a {@link News} instance.
+     */
+    public News build() {
+      return news;
+    }
+
+    /**
+     * Sets mandatory flag of the {@link News} instance to build.
+     * @param mandatory the mandatory flag.
+     * @return itself.
+     */
+    public Builder setMandatory(boolean mandatory) {
+      news.setMandatory(mandatory);
+      return this;
+    }
+
+    /**
+     * Sets ticker flag of the {@link News} instance to build.
+     * @param ticker the ticker flag.
+     * @return itself.
+     */
+    public Builder setTicker(boolean ticker) {
+      news.setTicker(ticker);
+      return this;
+    }
+
+    /**
+     * Sets important flag of the {@link News} instance to build.
+     * @param important the important flag.
+     * @return itself.
+     */
+    public Builder setImportant(boolean important) {
+      news.setImportant(important);
+      return this;
+    }
+
+    /**
+     * Sets the visibility period of the {@link News} instance to build.
+     * @param period the visibility period.
+     * @return itself.
+     */
+    public Builder setVisibilityPeriod(Period period) {
+      news.setVisibilityPeriod(period);
+      return this;
+    }
+
+    /**
+     * Sets the keywords of the {@link News} instance to build.
+     * @param keywords the keywords of the news.
+     * @return itself.
+     */
+    public Builder setKeywords(final String keywords) {
+      news.setKeywords(keywords);
+      return this;
+    }
+
+    /**
+     * Sets the given title and description of the news to build.
+     * @param title the title of the news.
+     * @param description the description of the news.
+     * @return itself.
+     */
+    public Builder setTitleAndDescription(final String title, final String description) {
+      news.setTitle(title);
+      news.setDescription(description);
+      return this;
+    }
+  }
 }