fix: use paymenthash instead of id for print

lnbits · Jan 2, 2025 · 3231d8f · 3231d8f
1 parent 8c88be1
commit 3231d8f
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/crud.py b/crud.py
@@ -24,10 +24,10 @@ async def update_print(_print: Print, conn: Optional[Connection] = None) -> Prin
     return _print
 
 
-async def get_print(print_id: str) -> Optional[Print]:
+async def get_print(payment_hash: str) -> Optional[Print]:
     return await db.fetchone(
-        "SELECT * FROM pay2print.print WHERE id = :id",
-        {"id": print_id},
+        "SELECT * FROM pay2print.print WHERE payment_hash = :payment_hash",
+        {"payment_hash": payment_hash},
         Print,
     )
 

diff --git a/models.py b/models.py
@@ -44,7 +44,6 @@ class CreatePrinter(BaseModel):
 
 
 class Print(BaseModel):
-    id: str = Field(default_factory=urlsafe_short_hash)
     printer: str
     payment_hash: str
     file: str

diff --git a/templates/pay2print/photo.html b/templates/pay2print/photo.html
@@ -42,7 +42,9 @@ <h2 class="text-h5">
         autoplay
         style="display: none"
       ></video>
-      <h3 v-if="!webcam" class="text-h6 text-center">Device does not support camera!</h3>
+      <h3 v-if="!webcam" class="text-h6 text-center">
+        Device does not support camera!
+      </h3>
       <p class="text-center text-negative" v-text="webcamError"></p>
       <canvas ref="canvas" width="0" height="0"></canvas>
       <q-card-section