0040598: Stop using Vector<Object> when only single type AttachContent is used

diff -r c51687516a3d src/org/openbravo/erpCommon/utility/reporting/printing/PrintController.java
--- a/src/org/openbravo/erpCommon/utility/reporting/printing/PrintController.java	Thu Apr 11 11:32:25 2019 +0200
+++ b/src/org/openbravo/erpCommon/utility/reporting/printing/PrintController.java	Thu Apr 11 15:09:08 2019 +0200
@@ -343,7 +343,7 @@
 
         } else if (vars.commandIn("DEL")) {
           final String documentToDelete = vars.getStringParameter("idToDelete");
-          final Vector<Object> vector = (Vector<Object>) request.getSession().getAttribute("files");
+          final Vector<AttachContent> vector = (Vector<AttachContent>) request.getSession().getAttribute("files");
           request.getSession().setAttribute("files", vector);
 
           seekAndDestroy(vector, documentToDelete);
@@ -414,7 +414,7 @@
               }
               final String senderAddress = vars.getStringParameter("fromEmail");
               sendDocumentEmail(report, vars,
-                  (Vector<Object>) request.getSession().getAttribute("files"), documentData,
+                  (Vector<AttachContent>) request.getSession().getAttribute("files"), documentData,
                   senderAddress, checks, documentType);
               nrOfEmailsSend++;
             }
@@ -660,14 +660,9 @@
     }
   }
 
-  /**
-   * 
-   * @param vector
-   * @param documentToDelete
-   */
-  private void seekAndDestroy(Vector<Object> vector, String documentToDelete) {
+  private void seekAndDestroy(Vector<AttachContent> vector, String documentToDelete) {
     for (int i = 0; i < vector.size(); i++) {
-      final AttachContent content = (AttachContent) vector.get(i);
+      final AttachContent content = vector.get(i);
       if (content.id.equals(documentToDelete)) {
         vector.remove(i);
         break;
@@ -697,7 +692,7 @@
     return null;
   }
 
-  void sendDocumentEmail(Report report, VariablesSecureApp vars, Vector<Object> object,
+  void sendDocumentEmail(Report report, VariablesSecureApp vars, Vector<AttachContent> vector,
       PocData documentData, String senderAddress, HashMap<String, Boolean> checks,
       DocumentType documentType) throws IOException, ServletException {
     final String attachmentFileLocation = report.getTargetLocation();
@@ -795,10 +790,9 @@
     List<File> attachments = new ArrayList<>();
     attachments.add(new File(attachmentFileLocation));
 
-    if (object != null) {
-      final Vector<Object> vector = object;
+    if (vector != null) {
       for (int i = 0; i < vector.size(); i++) {
-        final AttachContent objContent = (AttachContent) vector.get(i);
+        final AttachContent objContent = vector.get(i);
         final File file = prepareFile(objContent, ourReference);
         attachments.add(file);
       }
@@ -920,7 +914,7 @@
     XmlDocument xmlDocument = null;
     PocData[] pocData = getContactDetails(documentType, strDocumentId);
     @SuppressWarnings("unchecked")
-    Vector<java.lang.Object> vector = (Vector<java.lang.Object>) request.getSession()
+    Vector<AttachContent> vector = (Vector<AttachContent>) request.getSession()
         .getAttribute("files");
 
     final String[] hiddenTags = getHiddenTags(pocData, vector, vars, checks);
@@ -939,7 +933,7 @@
 
     boolean isTheFirstEntry = false;
     if (vector == null) {
-      vector = new Vector<java.lang.Object>(0);
+      vector = new Vector<>(0);
       isTheFirstEntry = true;
     }
 
@@ -1043,7 +1037,7 @@
     final boolean onlyOneAttachedDoc = onlyOneAttachedDocs(reports);
     final Map<String, PocData> customerMap = new HashMap<String, PocData>();
     final Map<String, PocData> salesRepMap = new HashMap<String, PocData>();
-    final Vector<Object> cloneVector = new Vector<Object>();
+    final Vector<AttachContent> cloneVector = new Vector<>();
     boolean allTheDocsCompleted = true;
     for (final PocData documentData : pocData) {
       // Map used to count the different users
@@ -1327,7 +1321,7 @@
   /**
    * @author gmauleon
    */
-  private String[] getHiddenTags(PocData[] pocData, Vector<Object> vector, VariablesSecureApp vars,
+  private String[] getHiddenTags(PocData[] pocData, Vector<AttachContent> vector, VariablesSecureApp vars,
       HashMap<String, Boolean> checks) {
     String[] discard;
     final Map<String, PocData> customerMap = new HashMap<String, PocData>();