[ 
http://opencast.jira.com/browse/MH-7649?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=26366#comment-26366
 ] 

Gurvinder Singh edited comment on MH-7649 at 5/1/11 7:11 AM:
-------------------------------------------------------------

Forget to add the patch. So here it is 

Index: 
modules/matterhorn-distribution-service-youtube/src/main/java/org/opencastproject/distribution/youtube/YoutubeDistributionService.java
===================================================================
--- 
modules/matterhorn-distribution-service-youtube/src/main/java/org/opencastproject/distribution/youtube/YoutubeDistributionService.java
      (revision 10352)
+++ 
modules/matterhorn-distribution-service-youtube/src/main/java/org/opencastproject/distribution/youtube/YoutubeDistributionService.java
      (working copy)
@@ -130,7 +130,7 @@
 
     // create the scheduler using file system store
     String directory_name = cc.getBundleContext().getProperty("youtube.task");
-    if (directory_name == null || directory_name.equals("")) {
+    if ((directory_name == null) || ("".equals(directory_name))) {
       directory_name = "/tmp/youtube";
     }
     logger.info("Task file directory: {}", directory_name);
@@ -417,8 +417,11 @@
    * @see 
org.opencastproject.job.api.AbstractJobProducer#process(org.opencastproject.job.api.Job,
 java.lang.String, java.util.List)
    */
   @Override
-  protected String process(Job job, String operation, List<String> arguments) 
throws Exception {
+  protected String process(Job job) throws Exception {
     Operation op = null;
+    String operation = job.getOperation();
+    List<String> arguments = job.getArguments();
+
     try {
       op = Operation.valueOf(operation);
       String mediapackageId = arguments.get(0);
Index: 
modules/matterhorn-distribution-service-youtube/src/main/java/org/opencastproject/deliver/youtube/YouTubeRemoveAction.java
===================================================================
--- 
modules/matterhorn-distribution-service-youtube/src/main/java/org/opencastproject/deliver/youtube/YouTubeRemoveAction.java
  (revision 10352)
+++ 
modules/matterhorn-distribution-service-youtube/src/main/java/org/opencastproject/deliver/youtube/YouTubeRemoveAction.java
  (working copy)
@@ -83,7 +83,7 @@
    */
 
   public void validate() throws InvalidException {
-    if (publish_task == null || publish_task.equals(""))
+    if ((publish_task == null) || ("".equals(publish_task)))
       throw new InvalidException("Missing publish task");
   }
 
Index: 
modules/matterhorn-distribution-service-youtube/src/main/java/org/opencastproject/deliver/youtube/YouTubeDeliveryAction.java
===================================================================
--- 
modules/matterhorn-distribution-service-youtube/src/main/java/org/opencastproject/deliver/youtube/YouTubeDeliveryAction.java
        (revision 10352)
+++ 
modules/matterhorn-distribution-service-youtube/src/main/java/org/opencastproject/deliver/youtube/YouTubeDeliveryAction.java
        (working copy)
@@ -302,7 +302,7 @@
       status(status_message);
       log(status_message);
 
-      if (domain.equals("yt:validation"))
+      if ("yt:validation".equals(domain))
         return new FailedException(except);
       else
         return new RetryException("Entry exception", except);
@@ -324,33 +324,33 @@
    */
 
   private String decodeYouTubeError(String domain, String code, String 
location) {
-    if (domain.equals("yt:validation")) {
-      if (code.equals("required"))
+    if ("yt:validation".equals(domain)) {
+      if ("required".equals(code))
         return "Missing: " + decodeLocation(location);
-      if (code.equals("deprecated"))
+      if ("deprecated".equals(code))
         return "Invalid: " + decodeLocation(location);
-      if (code.equals("invalid_format"))
+      if ("invalid_format".equals(code))
         return "Format invalid: " + decodeLocation(location);
-      if (code.equals("invalid_character"))
+      if ("invalid_character".equals(code))
         return "Invalid character: " + decodeLocation(location);
-      if (code.equals("too_long"))
+      if ("too_long".equals(code))
         return "Too long: " + decodeLocation(location);
       else
         return "Validation error: " + decodeLocation(location);
-    } else if (domain.equals("yt:quota")) {
-      if (code.equals("too_many_recent_calls"))
+    } else if ("yt:quota".equals(domain)) {
+      if ("too_many_recent_calls".equals(code))
         return "Too many uploads";
-      else if (code.equals("too_many_entries"))
+      else if ("too_many_entries".equals(code))
         return "Too many entries";
       else
         return "Quota exceeded";
-    } else if (domain.equals("yt:authentication")) {
-      if (code.equals("InvalidToken"))
+    } else if ("yt:authentication".equals(domain)) {
+      if ("InvalidToken".equals(code))
         return "Invalid access to service";
-      if (code.equals("TokenExpired"))
+      if ("TokenExpired".equals(code))
         return "Service session expired";
-    } else if (domain.equals("yt:service")) {
-      if (code.equals("disabled_in_maintenance_mode"))
+    } else if ("yt:service".equals(domain)) {
+      if ("disabled_in_maintenance_mode".equals(code))
         return "Service temporarily unavailable";
     }
     return "Delivery failed [3]";
Index: 
modules/matterhorn-distribution-service-youtube/src/main/resources/OSGI-INF/distribution-service-youtube.xml
===================================================================
--- 
modules/matterhorn-distribution-service-youtube/src/main/resources/OSGI-INF/distribution-service-youtube.xml
        (revision 10352)
+++ 
modules/matterhorn-distribution-service-youtube/src/main/resources/OSGI-INF/distribution-service-youtube.xml
        (working copy)
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <scr:component xmlns:scr="http://www.osgi.org/xmlns/scr/v1.1.0";
-  name="org.opencastproject.distribution.youtube.DistributionService" 
immediate="true" activate="activate" deactivate="deactivate">
+  name="org.opencastproject.distribution.youtube.DistributionService" 
immediate="true" activate="activate">
   <implementation 
class="org.opencastproject.distribution.youtube.YoutubeDistributionService" />
   <property name="service.description" value="Distribution Service (Youtube)" 
/>
   <property name="distribution.channel" value="youtube" />
Index: pom.xml
===================================================================
--- pom.xml     (revision 10352)
+++ pom.xml     (working copy)
@@ -92,11 +92,11 @@
         <module>modules/matterhorn-distribution-service-api</module>
         <module>modules/matterhorn-distribution-service-download</module>
         <module>modules/matterhorn-distribution-service-streaming</module>
-<!--
+
         <module>modules/matterhorn-distribution-service-common</module>
-        <module>modules/matterhorn-distribution-service-itunesu</module>
+<!--        <module>modules/matterhorn-distribution-service-itunesu</module> 
-->
         <module>modules/matterhorn-distribution-service-youtube</module>
--->
+
         <module>modules/matterhorn-dublincore</module>
         <module>modules/matterhorn-kernel</module>
         <module>modules/matterhorn-metadata-api</module>
@@ -120,10 +120,10 @@
         
<module>modules/matterhorn-distribution-service-download-remote</module>
         
<module>modules/matterhorn-distribution-service-streaming-remote</module>
 
-<!--
-        <module>modules/matterhorn-distribution-service-itunesu-remote</module>
+
+<!--        
<module>modules/matterhorn-distribution-service-itunesu-remote</module> -->
         <module>modules/matterhorn-distribution-service-youtube-remote</module>
--->
+
         <module>modules/matterhorn-dublincore</module>
         <module>modules/matterhorn-kernel</module>
         <module>modules/matterhorn-metadata-api</module>
@@ -174,11 +174,10 @@
         
<module>modules/matterhorn-distribution-service-download-remote</module>
         
<module>modules/matterhorn-distribution-service-streaming-remote</module>
 
-<!--
-        <module>modules/matterhorn-distribution-service-itunesu-remote</module>
+<!--        
<module>modules/matterhorn-distribution-service-itunesu-remote</module> -->
         <module>modules/matterhorn-distribution-service-youtube-remote</module>
--->
 
+
         <module>modules/matterhorn-dublincore</module>
         <module>modules/matterhorn-kernel</module>
         <module>modules/matterhorn-lti</module>


      was (Author: gurvindersingh):
    Forget to add the patch. So here it is 

Index: 
modules/matterhorn-distribution-service-youtube/src/main/resources/OSGI-INF/distribution-service-youtube.xml
===================================================================
--- 
modules/matterhorn-distribution-service-youtube/src/main/resources/OSGI-INF/distribution-service-youtube.xml
        (revision 10352)
+++ 
modules/matterhorn-distribution-service-youtube/src/main/resources/OSGI-INF/distribution-service-youtube.xml
        (working copy)
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<scr:component xmlns:scr="http://www.osgi.org/xmlns/scr/v1.1.0";
-  name="org.opencastproject.distribution.youtube.DistributionService" 
immediate="true" activate="activate" deactivate="deactivate">
+<scr:components xmlns:scr="http://www.osgi.org/xmlns/scr/v1.1.0";>
+<scr:component 
name="org.opencastproject.distribution.youtube.DistributionService" 
immediate="true" activate="activate">
   <implementation 
class="org.opencastproject.distribution.youtube.YoutubeDistributionService" />
   <property name="service.description" value="Distribution Service (Youtube)" 
/>
   <property name="distribution.channel" value="youtube" />
@@ -10,5 +10,25 @@
   </service>
   <reference name="WORKSPACE" 
interface="org.opencastproject.workspace.api.Workspace"
     cardinality="1..n" policy="static" bind="setWorkspace" />
-  <reference name="remoteServiceManager" 
interface="org.opencastproject.serviceregistry.api.ServiceRegistry" 
cardinality="1..1" policy="static" bind="setServiceRegistry" />
+  <reference name="serviceRegistry" 
interface="org.opencastproject.serviceregistry.api.ServiceRegistry" 
cardinality="1..1" policy="static" bind="setServiceRegistry" />
 </scr:component>
+
+<scr:component 
name="org.opencastproject.distribution.youtube.endpoint.YouTubeDistributionRestService"
 immediate="true" activate="activate">
+  <implementation 
class="org.opencastproject.distribution.youtube.endpoint.YouTubeDistributionRestService"
 />
+  <property name="service.description" value="YouTube Distribution REST 
Endpoint" />
+
+  <property name="opencast.service.type" 
value="org.opencastproject.distribution.youtube" />
+  <property name="opencast.service.path" value="/distribution/youtube" />
+  <property name="opencast.service.jobproducer" value="true" />
+
+  <service>
+    <provide 
interface="org.opencastproject.distribution.youtube.endpoint.YouTubeDistributionRestService"
 />
+  </service>
+
+  <reference name="distributionService" 
interface="org.opencastproject.distribution.api.DistributionService"
+    cardinality="1..1" policy="static" bind="setService" 
target="(distribution.channel=youtube)" />
+  <reference name="serviceRegistry" 
interface="org.opencastproject.serviceregistry.api.ServiceRegistry"
+      cardinality="1..1" policy="static" bind="setServiceRegistry" />
+</scr:component>
+
+</scr:components>
Index: pom.xml
===================================================================
--- pom.xml     (revision 10345)
+++ pom.xml     (working copy)
@@ -92,11 +92,11 @@
         <module>modules/matterhorn-distribution-service-api</module>
         <module>modules/matterhorn-distribution-service-download</module>
         <module>modules/matterhorn-distribution-service-streaming</module>
-<!--
+
         <module>modules/matterhorn-distribution-service-common</module>
-        <module>modules/matterhorn-distribution-service-itunesu</module>
+<!--    <module>modules/matterhorn-distribution-service-itunesu</module> -->
         <module>modules/matterhorn-distribution-service-youtube</module>
--->
+
         <module>modules/matterhorn-dublincore</module>
         <module>modules/matterhorn-kernel</module>
         <module>modules/matterhorn-metadata-api</module>
@@ -121,9 +121,9 @@
         
<module>modules/matterhorn-distribution-service-streaming-remote</module>
 
 <!--
-        <module>modules/matterhorn-distribution-service-itunesu-remote</module>
+        
<module>modules/matterhorn-distribution-service-itunesu-remote</module> -->
         <module>modules/matterhorn-distribution-service-youtube-remote</module>
--->
+
         <module>modules/matterhorn-dublincore</module>
         <module>modules/matterhorn-kernel</module>
         <module>modules/matterhorn-metadata-api</module>
@@ -175,10 +175,10 @@
         
<module>modules/matterhorn-distribution-service-streaming-remote</module>
 
 <!--
-        <module>modules/matterhorn-distribution-service-itunesu-remote</module>
+        
<module>modules/matterhorn-distribution-service-itunesu-remote</module> -->
         <module>modules/matterhorn-distribution-service-youtube-remote</module>
--->
 
+
         <module>modules/matterhorn-dublincore</module>
         <module>modules/matterhorn-kernel</module>
         <module>modules/matterhorn-lti</module>


  
> youtube distribution error no workflow handler
> ----------------------------------------------
>
>                 Key: MH-7649
>                 URL: http://opencast.jira.com/browse/MH-7649
>             Project: Matterhorn Project
>          Issue Type: Bug
>          Components: Distribution
>    Affects Versions: 1.1 
>            Reporter: Gurvinder Singh
>


--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        
_______________________________________________
Matterhorn mailing list
[email protected]
http://lists.opencastproject.org/mailman/listinfo/matterhorn


To unsubscribe please email
[email protected]
_______________________________________________

Reply via email to