Ignore other video subscribe options when RID is set (#248)

starwarfan · jianjunz · commit b4d2877bf623 · 2019-09-18T08:43:20.000+08:00
* Ignore other video subscribe options when RID is set

* Update comments
diff --git a/src/sdk/conference/channel.js b/src/sdk/conference/channel.js
@@ -339,6 +339,9 @@ export class ConferencePeerConnectionChannel extends EventDispatcher {
       }
       if (options.video.rid) {
         mediaOptions.video.simulcastRid = options.video.rid;
+        // Ignore other settings when RID set.
+        delete mediaOptions.video.parameters;
+        options.video = true;
       }
     } else {
       mediaOptions.video = false;

Original file line number	Diff line number	Diff line change
`@@ -339,6 +339,9 @@ export class ConferencePeerConnectionChannel extends EventDispatcher {`
`339`	`339`	`}`
`340`	`340`	`if (options.video.rid) {`
`341`	`341`	`mediaOptions.video.simulcastRid = options.video.rid;`
	`342`	`+ // Ignore other settings when RID set.`
	`343`	`+ delete mediaOptions.video.parameters;`
	`344`	`+ options.video = true;`
`342`	`345`	`}`
`343`	`346`	`} else {`
`344`	`347`	`mediaOptions.video = false;`