light9 Changeset - e9239995eb57

Changeset - e9239995eb57

Parent rev.

Child rev.

[Not reviewed]

default

0 2 0

drewp@bigasterisk.com - 20 months ago 2023-05-30 06:08:15
drewp@bigasterisk.com

fix effect new/clear commands

2 files changed with 40 insertions and 20 deletions:

light9/live/Effect.ts

light9/live/Light9DeviceSettings.ts

0 comments (0 inline, 0 general)

light9/live/Effect.ts

➞

Show inline comments

@@ @@ -25,40 +25,51 @@ function valuePred(graph: SyncedGraph, a @@
     return U(":value");
   } else {
     return U(":value");
+  }
+}
 function effContext(graph: SyncedGraph, uri: NamedNode): NamedNode {
   return graph.Uri(uri.value.replace("light9.bigasterisk.com/effect", `light9.bigasterisk.com/show/${shortShow}/effect`));
+}
 export function newEffect(graph: SyncedGraph): NamedNode {
   // wrong- this should be our editor's scratch effect, promoted to a
   // real one when you name it.
   const uri = graph.nextNumberedResource(graph.Uri("http://light9.bigasterisk.com/effect/effect"));
   const effect = new Effect(graph, uri);
   const U = graph.U();
   const ctx = effContext(graph, uri);
   const quad = (s: Quad_Subject, p: Quad_Predicate, o: Quad_Object) => graph.Quad(s, p, o, ctx);
   const addQuads = [
     quad(uri, U("rdf:type"), U(":Effect")),
     quad(uri, U("rdfs:label"), graph.Literal(uri.value.replace(/.*\//, ""))),
     quad(uri, U(":publishAttr"), U(":strength")),
   ];
   const patch = new Patch([], addQuads);
   log("init new effect", patch);
   graph.applyAndSendPatch(patch);
   return effect.uri;
+}
 // effect settings data; r/w sync with the graph
 export class Effect {
   private settings: Array<{ device: NamedNode; deviceAttr: NamedNode; setting: NamedNode; value: ControlValue }> = [];
   private ctxForEffect: NamedNode;
   settingsChanged: SubEvent<void> = new SubEvent();
   constructor(
     public graph: SyncedGraph,
     public uri: NamedNode // called if the graph changes our values and not when the caller uses edit()
   ) {
-    this.ctxForEffect = this.graph.Uri(this.uri.value.replace("light9.bigasterisk.com/effect", `light9.bigasterisk.com/show/${shortShow}/effect`));
+    this.ctxForEffect = effContext(this.graph, this.uri);
     graph.runHandler(this.rebuildSettingsFromGraph.bind(this), `effect sync ${uri.value}`);
+  }
   addNewEffectToGraph() {
     const U = this.graph.U();
     const quad = (s: Quad_Subject, p: Quad_Predicate, o: Quad_Object) => this.graph.Quad(s, p, o, this.ctxForEffect);
     const addQuads = [
       quad(this.uri, U("rdf:type"), U(":Effect")),
       quad(this.uri, U("rdfs:label"), this.graph.Literal(this.uri.value.replace(/.*\//, ""))),
       quad(this.uri, U(":publishAttr"), U(":strength")),
     ];
     const patch = new Patch([], addQuads);
     log("init new effect", patch);
     this.settings = [];
     this.graph.applyAndSendPatch(patch);
+  }
   rebuildSettingsFromGraph(patch?: Patch) {
     const U = this.graph.U();
     if (patch && !patch.containsAnyPreds([U(":setting"), U(":device"), U(":deviceAttr")])) {
       // that's an approx list of preds , but it just means we'll miss some pathological settings edits
       //   return;
+    }
@@ @@ -183,12 +194,18 @@ export class Effect { @@
     for (let q of this.graph.subjectStatements(effectSetting)) {
       toDel.push(q);
+    }
     return new Patch(toDel, []);
+  }
 clearAllSettings() {
   for (let s of this.settings) {
     this.graph.applyAndSendPatch(this.removeEffectSetting(s.setting))
+  }
+}
   private nodeForValue(value: ControlValue): NamedNode | Literal {
     if (value === null) {
       throw new Error("no value");
+    }
     if (isUri(value)) {
       return value;

light9/live/Light9DeviceSettings.ts

➞

Show inline comments

@@ @@ -3,13 +3,13 @@ import { css, html, LitElement, Property @@
 import { customElement, property } from "lit/decorators.js";
 import { NamedNode } from "n3";
 import { sortBy, uniq } from "underscore";
 import { Patch } from "../web/patch";
 import { getTopGraph } from "../web/RdfdbSyncedGraph";
 import { SyncedGraph } from "../web/SyncedGraph";
 import { Effect } from "./Effect";
+import { Effect, newEffect } from "./Effect";
 export { EditChoice } from "../web/EditChoice";
 export { Light9DeviceControl as Light9LiveDeviceControl } from "./Light9DeviceControl";
 const log = debug("settings");
 @customElement("light9-device-settings")
 export class Light9DeviceSettings extends LitElement {
@@ @@ -77,14 +77,18 @@ export class Light9DeviceSettings extend @@
       this.setEffectFromUrl();
     });
+  }
   onEffectChoice2(ev: CustomEvent) {
     const uri = ev.detail.newValue as NamedNode;
     this.setCurrentEffect(uri);
+  }
   setCurrentEffect(uri: NamedNode) {
     if (uri === null) {
       this.currentEffect = null;
       // todo: wipe the UI settings
     } else {
       this.currentEffect = new Effect(this.graph, uri);
+    }
+  }
   updated(changedProperties: PropertyValues<this>) {
@@ @@ -137,14 +141,13 @@ export class Light9DeviceSettings extend @@
     u.searchParams.set("effect", effectStr); // this escapes : and / and i wish it didn't
     window.history.replaceState({}, "", u.href);
     log("wrote new url", u.href);
+  }
   newEffect() {
-    // this.effectChoice = this.graphToControls.newEffect();
+    this.setCurrentEffect(newEffect(this.graph));
+  }
   clearAll() {
     // clears the effect!
     return; //this.graphToControls.emptyEffect();
     this.currentEffect?.clearAllSettings()
+  }
+}

0 comments (0 inline, 0 general)