refactored code for better readability
[meviz.git] / src / main / java / eu / svjatoslav / meviz / htmlindexer / layouts / MixedLayout.java
index 915c7c8..4e6b5bf 100755 (executable)
@@ -21,6 +21,7 @@ import eu.svjatoslav.meviz.htmlindexer.Constants;
 import eu.svjatoslav.meviz.htmlindexer.IndexingContext;
 import eu.svjatoslav.meviz.htmlindexer.Layout;
 import eu.svjatoslav.meviz.htmlindexer.metadata.Dimension;
+import eu.svjatoslav.meviz.htmlindexer.metadata.DirectoryMetadata;
 import eu.svjatoslav.meviz.htmlindexer.metadata.fileTypes.AbstractFile;
 import eu.svjatoslav.meviz.htmlindexer.metadata.fileTypes.GeneralFile;
 import eu.svjatoslav.meviz.htmlindexer.metadata.fileTypes.Picture;
@@ -39,26 +40,13 @@ public class MixedLayout implements Layout {
 
        public static final String encoding = "UTF-8";
 
-       int currentColumn = 0;
-
-       int horizontalColumnsLimit = 2;
-
        private AbstractIndexer indexer;
 
        private IndexingContext indexingContext;
 
-       public void columnAdded() {
-               currentColumn++;
-               if (currentColumn > horizontalColumnsLimit) {
-
-                       currentColumn = 0;
-
-                       filesHtml.append("</TR>\n");
-               }
-       }
-
        @Override
-       public void enlistDirectory(final AbstractFile directory) {
+       public void enlistDirectory(final AbstractFile directory,
+                       final DirectoryMetadata parentDirectoryMetadata) {
                dirHtml.append("<TR>");
                dirHtml.append("<TD><b><a href=\""
                                + indexer.getDirectoryUrl(directory, indexingContext)
@@ -68,136 +56,119 @@ public class MixedLayout implements Layout {
        }
 
        @Override
-       public void enlistFile(final AbstractFile file) {
-               finishRow();
+       public void enlistFile(final AbstractFile file,
+                       final DirectoryMetadata parentDirectoryMetadata) {
 
                final String fullFileName = file.fileName;
 
-               filesHtml.append("<TR>\n");
-               filesHtml.append("    <TD colspan=\""
-                               + String.valueOf(horizontalColumnsLimit)
-                               + "\"><a href=\""
+               filesHtml.append("<div style=\"clear: both;\"><a href=\""
                                + indexingContext.getGlobalUrl()
                                + UrlParamEncoder.encode(indexingContext.getLocalUrl() + "/"
                                                + fullFileName) + "\">" + fullFileName + "</a>\n");
                filesHtml.append("&nbsp;&nbsp;("
                                + FilePathParser.getFileSizeDescription(file.getFileLength())
-                               + ")</TD>\n");
-               filesHtml.append("</TR>\n");
+                               + ")</div><br/>\n");
 
        }
 
        @Override
-       public void enlistImage(final AbstractFile abstractFile) {
+       public void enlistImage(final Picture picture,
+                       final DirectoryMetadata parentDirectoryMetadata) {
                try {
+                       enlistImageInternal(picture);
+               } catch (final Exception e) {
+                       logger.error(
+                                       "Error generating thumbnails for image file: "
+                                                       + e.toString()
+                                                       + ", thumbnail generation skipped, enlisting as ordinary file instead.",
+                                       e);
+                       enlistFile(picture, parentDirectoryMetadata);
+               }
+       }
 
-                       final Picture picture = (Picture) abstractFile;
-
-                       final Dimension imageSize = picture.getDimensions();
-
-                       final List<Dimension> recommendedDimensions = Picture
-                                       .getThumbnailDimensions(imageSize);
-
-                       Dimension listingDimension = new Dimension(500, 500);
-
-                       if (listingDimension.width > imageSize.width)
-                               if (listingDimension.height > imageSize.height)
-                                       listingDimension = imageSize;
-
-                       final String listingThumbnailUrl = indexer.getThumbnailUrl(picture,
-                                       listingDimension, indexingContext);
+       private String enlistImageDimensions(final Picture picture) {
+               final StringBuilder html = new StringBuilder();
+
+               final List<Dimension> recommendedDimensions = Picture
+                               .getThumbnailDimensions(picture.getDimensions());
+
+               int count = 0;
+               for (final Dimension dimension : recommendedDimensions) {
+                       count++;
+                       if (count > 1)
+                               html.append(", ");
+                       html.append("<a href=\""
+                                       + indexer.getThumbnailUrl(picture, dimension,
+                                                       indexingContext) + "\">");
+                       html.append(dimension.width + "x" + dimension.height);
+                       html.append("</a>");
+               }
 
-                       startcolumn();
+               return html.toString();
+       }
 
-                       filesHtml.append("<TD><center><a href=\""
-                                       + indexingContext.getGlobalUrl()
-                                       + UrlParamEncoder.encode(indexingContext.getLocalUrl()
-                                                       + "/" + picture.fileName) + "\">\n");
+       private void enlistImageInternal(final Picture picture) {
 
-                       filesHtml.append("<img border=\"0\" src=\"" + listingThumbnailUrl
-                                       + "\"/></a>\n");
-                       filesHtml.append("<br/><b>"
-                                       + FilePathParser
-                                                       .getFileNameWithoutExtension(picture.fileName)
-                                       + "</b><br/>(");
+               filesHtml.append("<div style=\"float:left; margin: 10px;\">");
 
-                       int count = 0;
-                       for (final Dimension dimension : recommendedDimensions) {
-                               count++;
-                               if (count > 1)
-                                       filesHtml.append(", ");
-                               filesHtml.append("<a href=\""
-                                               + indexer.getThumbnailUrl(picture, dimension,
-                                                               indexingContext) + "\">");
-                               filesHtml.append(dimension.width + "x" + dimension.height);
-                               filesHtml.append("</a>");
-                       }
+               // clickable image thumbnail
+               filesHtml.append("<a href=\""
+                               + indexingContext.getGlobalUrl()
+                               + UrlParamEncoder.encode(indexingContext.getLocalUrl() + "/"
+                                               + picture.fileName) + "\">\n");
+               filesHtml.append("<img border=\"0\" src=\""
+                               + getImageThumbnailUrl(picture) + "\"/></a><br/>\n");
 
-                       filesHtml.append(")</TD>\n");
+               // file name
+               filesHtml.append("<b>"
+                               + FilePathParser.getFileNameWithoutExtension(picture.fileName)
+                               + "</b><br/>");
 
-                       columnAdded();
+               // links to thumbnails of various sizes
+               filesHtml.append("(" + enlistImageDimensions(picture) + ")\n");
 
-               } catch (final Exception e) {
-                       logger.error(
-                                       "Error generating thumbnails for image file: "
-                                                       + e.toString()
-                                                       + ", thumbnail generation skipped, enlisting as ordinary file instead.",
-                                       e);
-                       enlistFile(abstractFile);
-               }
+               filesHtml.append("</div>");
        }
 
        /** {@inheritDoc} */
        @Override
-       public void enlistOgv(final AbstractFile abstractFile)
+       public void enlistOgv(final AbstractFile abstractFile,
+                       final DirectoryMetadata parentDirectoryMetadata)
                        throws UnsupportedEncodingException {
 
                final GeneralFile file = (GeneralFile) abstractFile;
 
-               finishRow();
+               filesHtml.append("<div style=\"margin: 30px; clear: both;\">");
 
+               // watchable video
                filesHtml
-                               .append("<TR style=\"background-color: #404040; text-align: center\">\n");
-               filesHtml.append("<TD style=\"text-align: center\" colspan=\""
-                               + String.valueOf(horizontalColumnsLimit) + "\">\n");
-
-               filesHtml.append("<video controls=\"controls\" tabindex=\"0\">\n");
+                               .append("<video preload=\"metadata\" controls=\"controls\" tabindex=\"0\">\n");
                filesHtml
                                .append("<source type=\"video/ogg\" codecs=\"theora, vorbis\" src=\""
                                                + indexingContext.getGlobalUrl()
                                                + UrlParamEncoder.encode(indexingContext.getLocalUrl()
                                                                + "/" + file.fileName) + "\">\n");
                filesHtml.append("</source>\n");
-
                filesHtml.append("</video>\n");
-
                filesHtml.append("<br/>\n");
 
+               // video title
                filesHtml
                                .append("Video: <b>"
                                                + FilePathParser
                                                                .getFileNameWithoutExtension(file.fileName)
                                                + "<b/>  &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;");
+
+               // video download link
                filesHtml.append("<a href=\""
                                + indexingContext.getGlobalUrl()
                                + UrlParamEncoder.encode(indexingContext.getLocalUrl() + "/"
-                                               + file.fileName) + "\">download as OGV</a>\n");
+                                               + file.fileName) + "\">download as OGV</a>");
 
-               filesHtml.append("</TD></TR>\n");
+               filesHtml.append("</div>\n");
 
-       }
-
-       public void finishRow() {
-
-               if (currentColumn == 0)
-                       return;
+               filesHtml.append("\n");
 
-               while (currentColumn <= horizontalColumnsLimit) {
-                       filesHtml.append("<TD></TD>");
-                       currentColumn++;
-               }
-               filesHtml.append("</TR>\n");
-               currentColumn = 0;
        }
 
        /** {@inheritDoc} */
@@ -232,32 +203,11 @@ public class MixedLayout implements Layout {
                        finalHtml
                                        .append("<CENTER><H1>" + galleryTitle + "</H1></CENTER>\n");
 
-               finalHtml.append("<CENTER><H2>");
-               for (final String p : path)
-                       finalHtml.append(" / " + p);
-               finalHtml.append("</H2></CENTER>\n");
-
-               // append directory listing
-               finalHtml.append("<TABLE>\n");
-
-               if (path.size() > 0) {
-                       finalHtml.append("<TR>");
-                       finalHtml.append("<TD><b><a href=\""
-                                       + indexer.getParentDirectoryUrl(indexingContext)
-                                       + "\"><h2> &lt;== parent directory</h2></a></b></TD>");
-                       finalHtml.append("</TR>\n");
-               }
-
-               finalHtml.append(dirHtml.toString());
-               finalHtml.append("\n<TR><TD><br></TD></TR>\n");
-               finalHtml.append("</TABLE>\n");
+               showCurrentPath(finalHtml);
 
-               // append file listing
+               listDirectories(finalHtml);
 
-               finishRow();
-               finalHtml.append("<TABLE width=\"1100\">\n");
                finalHtml.append(filesHtml.toString());
-               finalHtml.append("\n</TABLE>\n");
 
                if (doHtmlHeaderAndFooter)
                        finalHtml.append("</font></BODY></HTML>\n");
@@ -265,6 +215,19 @@ public class MixedLayout implements Layout {
                return finalHtml.toString();
        }
 
+       private String getImageThumbnailUrl(final Picture picture) {
+               final Dimension imageSize = picture.getDimensions();
+
+               Dimension listingDimension = new Dimension(1500, 450);
+
+               if (listingDimension.width > imageSize.width)
+                       if (listingDimension.height > imageSize.height)
+                               listingDimension = imageSize;
+
+               return indexer.getThumbnailUrl(picture, listingDimension,
+                               indexingContext);
+       }
+
        @Override
        public void init(final String galleryTitle, final List<String> path,
                        final AbstractIndexer indexer, final IndexingContext indexingContext) {
@@ -278,11 +241,27 @@ public class MixedLayout implements Layout {
                filesHtml = new StringBuffer();
        }
 
-       public void startcolumn() {
-               if (currentColumn == 0) {
-                       filesHtml.append("<TR>");
-                       currentColumn = 1;
+       private void listDirectories(final StringBuffer finalHtml) {
+               finalHtml.append("<TABLE>\n");
+
+               if (path.size() > 0) {
+                       finalHtml.append("<TR>");
+                       finalHtml.append("<TD><b><a href=\""
+                                       + indexer.getParentDirectoryUrl(indexingContext)
+                                       + "\"><h2> &lt;== parent directory</h2></a></b></TD>");
+                       finalHtml.append("</TR>\n");
                }
+
+               finalHtml.append(dirHtml.toString());
+               finalHtml.append("\n<TR><TD><br></TD></TR>\n");
+               finalHtml.append("</TABLE>\n");
+       }
+
+       private void showCurrentPath(final StringBuffer finalHtml) {
+               finalHtml.append("<CENTER><H2>");
+               for (final String p : path)
+                       finalHtml.append(" / " + p);
+               finalHtml.append("</H2></CENTER>\n");
        }
 
 }