diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideoCreateDialog.java b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideoCreateDialog.java index 48743d9268f..ef66b5843d4 100644 --- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideoCreateDialog.java +++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideoCreateDialog.java @@ -4,6 +4,7 @@ import java.util.Collections; import com.google.gwt.user.client.ui.Label; import com.sap.sailing.gwt.ui.client.StringMessages; +import com.sap.sse.common.media.MimeType; import com.sap.sse.gwt.client.media.VideoDTO; public class VideoCreateDialog extends VideoDialog { @@ -19,5 +20,6 @@ public class VideoCreateDialog extends VideoDialog { copyrightTextBox.setVisibleLength(50); lengthIntegerBox = createIntegerBox(null, 10); tagsListEditor.setValue(Collections.emptyList()); + setSelectedMimeType(MimeType.unknown); } } diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideoDialog.java b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideoDialog.java index caf6d7b76bf..a091018b7b8 100644 --- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideoDialog.java +++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideoDialog.java @@ -70,7 +70,6 @@ public abstract class VideoDialog extends DataEntryDialog { mimeTypeListBox.addItem(MimeType.ogg.name()); mimeTypeListBox.addItem(MimeType.ogv.name()); mimeTypeListBox.addItem(MimeType.qt.name()); - mimeTypeListBox.setSelectedIndex(0); videoURLAndUploadComposite = new URLFieldWithFileUpload(stringMessages); videoURLAndUploadComposite.addValueChangeHandler(new ValueChangeHandler() { @@ -100,6 +99,15 @@ public abstract class VideoDialog extends DataEntryDialog { return result; } + protected void setSelectedMimeType(MimeType mimeType) { + for(int i = 0; i < mimeTypeListBox.getVisibleItemCount(); i++) { + if(mimeTypeListBox.getItemText(0).equals(mimeType.name())) { + mimeTypeListBox.setSelectedIndex(i); + break; + } + } + } + private MimeType getSelectedMimeType() { MimeType result = null; int selectedIndex = mimeTypeListBox.getSelectedIndex(); diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideoEditDialog.java b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideoEditDialog.java index c528d410fa0..b8e63113d73 100644 --- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideoEditDialog.java +++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideoEditDialog.java @@ -22,5 +22,6 @@ public class VideoEditDialog extends VideoDialog { List tags = new ArrayList(); tags.addAll(video.getTags()); tagsListEditor.setValue(tags); + setSelectedMimeType(video.getMimeType()); } } diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideosListComposite.java b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideosListComposite.java index a2280a71863..b9ed5d13892 100644 --- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideosListComposite.java +++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/VideosListComposite.java @@ -85,8 +85,8 @@ public class VideosListComposite extends Composite { videoSelectionModel = new SingleSelectionModel(); videoListDataProvider = new ListDataProvider(); - videoTable = createImagesTable(); - videoTable.ensureDebugId("ImagesCellTable"); + videoTable = createVideosTable(); + videoTable.ensureDebugId("VideosCellTable"); videoTable.setVisible(false); videoSelectionModel.addSelectionChangeHandler(new SelectionChangeEvent.Handler() { @@ -105,7 +105,7 @@ public class VideosListComposite extends Composite { initWidget(mainPanel); } - private CellTable createImagesTable() { + private CellTable createVideosTable() { CellTable table = new CellTable(/* pageSize */10000, tableRes); videoListDataProvider.addDataDisplay(table); table.setWidth("100%"); @@ -132,19 +132,26 @@ public class VideosListComposite extends Composite { TextColumn createdAtDateColumn = new TextColumn() { @Override - public String getValue(VideoDTO image) { - return DateAndTimeFormatterUtil.formatDateAndTime(image.getCreatedAtDate()); + public String getValue(VideoDTO video) { + return DateAndTimeFormatterUtil.formatDateAndTime(video.getCreatedAtDate()); + } + }; + + TextColumn mimeTypeColumn = new TextColumn() { + @Override + public String getValue(VideoDTO video) { + return video.getMimeType() != null ? video.getMimeType().name() : ""; } }; SafeHtmlCell tagsCell = new SafeHtmlCell(); Column tagsColumn = new Column(tagsCell) { @Override - public SafeHtml getValue(VideoDTO image) { + public SafeHtml getValue(VideoDTO video) { SafeHtmlBuilder builder = new SafeHtmlBuilder(); - int tagsCount = image.getTags().size(); + int tagsCount = video.getTags().size(); int i = 1; - for (String tag : image.getTags()) { + for (String tag : video.getTags()) { builder.appendEscaped(tag); if (i < tagsCount) { builder.appendHtmlConstant(", "); @@ -163,12 +170,12 @@ public class VideosListComposite extends Composite { new VideoConfigImagesBarCell(stringMessages)); videoActionColumn.setFieldUpdater(new FieldUpdater() { @Override - public void update(int index, VideoDTO image, String value) { + public void update(int index, VideoDTO video, String value) { if (ImageConfigImagesBarCell.ACTION_REMOVE.equals(value)) { - videoListDataProvider.getList().remove(image); + videoListDataProvider.getList().remove(video); updateTableVisisbilty(); } else if (ImageConfigImagesBarCell.ACTION_EDIT.equals(value)) { - openEditVideoDialog(image); + openEditVideoDialog(video); } } }); @@ -179,6 +186,7 @@ public class VideosListComposite extends Composite { table.addColumn(shortVideoNameColumn, "Short name"); table.addColumn(titleColumn, stringMessages.title()); table.addColumn(createdAtDateColumn, "Created At"); + table.addColumn(mimeTypeColumn, stringMessages.mimeType()); table.addColumn(tagsColumn, "Tags"); table.addColumn(videoActionColumn, stringMessages.actions()); table.addColumnSortHandler(getVideoTableColumnSortHandler(videoListDataProvider.getList(), titleColumn, createdAtDateColumn)); @@ -237,9 +245,9 @@ public class VideosListComposite extends Composite { } @Override - public void ok(VideoDTO updatedImage) { + public void ok(VideoDTO updatedVideo) { videoListDataProvider.getList().remove(selectedVideo); - videoListDataProvider.getList().add(updatedImage); + videoListDataProvider.getList().add(updatedVideo); updateTableVisisbilty(); } });