[vlma-devel] [PATCH] MediasProgramAddFormController: Don't prefix channels names with [TNT], [SAT] or [VLMa].

Pierre d'Herbemont pdherbemont at videolan.org
Wed Apr 9 14:43:49 CEST 2008


This pollute a bit the playlist, and the info is already present in the SAP group. The provider (TNT/Sat) could probably be added to the meta data of the stream, in replacement.

This patch is probably a bit subjective. But rationally, meta data should go in meta data fields, not in the name of the stream.
---
 .../web/medias/MediasProgramAddFormController.java |    8 +++-----
 1 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/web/src/main/java/org/videolan/vlma/web/medias/MediasProgramAddFormController.java b/web/src/main/java/org/videolan/vlma/web/medias/MediasProgramAddFormController.java
index 492afc1..1fe4d2a 100644
--- a/web/src/main/java/org/videolan/vlma/web/medias/MediasProgramAddFormController.java
+++ b/web/src/main/java/org/videolan/vlma/web/medias/MediasProgramAddFormController.java
@@ -105,14 +105,12 @@ public class MediasProgramAddFormController extends SimpleFormController {
 		if (media.getClass() == VlSatChannel.class){
 			String country = new String(((VlSatChannel)media).getCountry());  
 			if ( country != "")
-				mediasProgramAdd.setSap("[SAT][VLMa][" + country + "] " + media.getName());
+				mediasProgramAdd.setSap("[" + country + "] " + media.getName());
 			else
-				mediasProgramAdd.setSap("[SAT][VLMa] " + media.getName());
+				mediasProgramAdd.setSap(media.getName());
 		}
-		else if (media.getClass() == VlTNTChannel.class)
-			mediasProgramAdd.setSap("[TNT][VLMa] " + media.getName());
 		else
-			mediasProgramAdd.setSap("[VLMa] " + media.getName());
+			mediasProgramAdd.setSap(media.getName());
 			
 		mediasProgramAdd.setMediaId(mediaId);
 		mediasProgramAdd.setData(data);
-- 
1.5.4.4.481.g5075



More information about the vlma-devel mailing list