Changeset 26236

Timestamp:

2012-09-24T10:43:05+12:00 (12 years ago)

Author:

jmt12

Message:

Limit the size of the keyframe preview field so that we can have a hardcoded lenght for the field in terrier.properties configuration without breaking things. I've started with maxlength=1024 - which should be around 80 thumbnails.

File:

• gs2-extensions/video-and-audio/trunk/src/opt/Terrier/VideoDocument.java (modified) (2 diffs)

gs2-extensions/video-and-audio/trunk/src/opt/Terrier/VideoDocument.java

@@ -67 +67 @@
   protected final String preview_width = ApplicationSetup.getProperty("ImageDocument.preview_width", "200");
 
+  protected int max_keyframe_images_length = 1024;
 
   /** Default constructor. **/
@@ -287 +288 @@
           else
           {
-            keyframe_images += "," + image_filename;
+            // Consider the maximum size of the preview images field
+            if ((keyframe_images.length() + image_filename.length() + 1) < this.max_keyframe_images_length)
+            {
+              keyframe_images += "," + image_filename;
+            }
+            // Break out of loop
+            else
+            {
+              i = files.length;
+            }
           }
         }