cleanup; better query; better readability

Author: Soxasora

api/paidAction/itemUpdate.js

@@ -14,16 +14,16 @@ export const paymentMethods = [
 
 export async function getCost ({ subName, id, boost = 0, uploadIds, bio }, { me, models }) {
   // the only reason updating items costs anything is when it has new uploads
-  // or more boost or is switch to a more expensive sub
+  // or more boost or is switching to a more expensive sub
   const old = await models.item.findUnique({ where: { id: parseInt(id) } })
   const { totalFeesMsats } = await uploadFees(uploadIds, { models, me })
   let cost = BigInt(totalFeesMsats) + satsToMsats(boost - old.boost)
+
   if (old.subName !== subName) {
-    const oldSub = await models.sub.findUnique({ where: { name: old.subName } })
-    const newSub = await models.sub.findUnique({ where: { name: subName } })
-    const oldCost = oldSub?.baseCost ?? 0
-    const newCost = newSub?.baseCost ?? 0
-    cost += satsToMsats(Math.max(0, newCost - oldCost)) // user will pay just the difference
+    const oldSub = await models.sub.findUnique({ where: { name: old.subName }, select: { baseCost: true } })
+    const newSub = await models.sub.findUnique({ where: { name: subName }, select: { baseCost: true } })
+    // user will pay just the difference
+    cost += satsToMsats(Math.max(0, (newSub?.baseCost ?? 0) - (oldSub?.baseCost ?? 0)))
   }
 
   if (cost > 0 && old.invoiceActionState && old.invoiceActionState !== 'PAID') {

fragments/items.js

@@ -35,6 +35,7 @@ export const ITEM_FIELDS = gql`
       meMuteSub
       meSubscription
       nsfw
+      baseCost
       replyCost
     }
     otsHash

pages/items/[id]/edit.js

@@ -20,8 +20,7 @@ export const getServerSideProps = getGetServerSideProps({
   notFound: data => !data.item
 })
 
-// TODO: cleanup
-const SUB_QUERY = gql`
+const SUB_BASECOST = gql`
   query Sub($name: String!) {
     sub(name: $name) {
       name
@@ -39,16 +38,8 @@ export default function PostEdit ({ ssrData }) {
   const { me } = useMe()
   const [sub, setSub] = useState(item.subName)
 
-  // TODO: cleanup
-  const { data: oldSubData } = useQuery(SUB_QUERY, {
-    variables: { name: item.subName },
-    skip: !item.subName
-  })
-
-  const { data: newSubData } = useQuery(SUB_QUERY, {
-    variables: { name: sub },
-    skip: !sub
-  })
+  // we need to fetch the new sub to calculate the cost difference
+  const { data: newSubData } = useQuery(SUB_BASECOST, { variables: { name: sub } })
 
   const [,, editThreshold] = useCanEdit(item)
 
@@ -68,7 +59,7 @@ export default function PostEdit ({ ssrData }) {
     itemType = 'BOUNTY'
   }
 
-  function editLineItems (oldSub, newSub) {
+  function editLineItems (newSub) {
     const existingBoostLineItem = item.boost
       ? {
           existingBoost: {
@@ -79,15 +70,24 @@ export default function PostEdit ({ ssrData }) {
           }
         }
       : undefined
+
+    const isSwitchingSub = item.subName !== newSub?.name
+    const subCostDifference = isSwitchingSub && {
+      ...postCommentBaseLineItems({
+        baseCost: Math.max(0, (newSub?.baseCost ?? 0) - (item?.sub?.baseCost ?? 0)),
+        me: !!me
+      })
+    }
+
     return {
-      ...(item.subName !== newSub?.name ? postCommentBaseLineItems({ baseCost: Math.max(0, newSub?.baseCost - oldSub?.baseCost), me: !!me }) : undefined),
+      ...subCostDifference,
       ...existingBoostLineItem
     }
   }
 
   return (
     <CenterLayout sub={sub}>
-      <FeeButtonProvider baseLineItems={editLineItems(oldSubData?.sub, newSubData?.sub)}>
+      <FeeButtonProvider baseLineItems={editLineItems(newSubData?.sub)}>
         <FormType item={item} editThreshold={editThreshold}>
           {!item.isJob &&
             <SubSelect