feat: amx article lenght and update upload status

Author: Leoglme

api/routes/articles.py

@@ -19,7 +19,13 @@
 from models.article import Article
 from models.image import Image as ImageModel
 from models.user import User
-from schemas.articles import ArticleUpdate, BulkIdsBody, SoldPatch, VintedBatchStartBody
+from schemas.articles import (
+    ARTICLE_TITLE_MAX_LEN_VINTED,
+    ArticleUpdate,
+    BulkIdsBody,
+    SoldPatch,
+    VintedBatchStartBody,
+)
 from services import article_service
 from services.combined_marketplace_service import CombinedMarketplaceService
 from services.ebay_background_service import EbayBackgroundService
@@ -70,6 +76,25 @@ def _parse_sold_at_iso(value: str | None) -> dt.datetime | None:
         ) from exc
 
 
+def _validate_create_title_or_raise(raw_title: str) -> str:
+    """Titre requis, longueur max alignée sur Vinted (espaces inclus après trim côté annonce)."""
+    t = raw_title.strip()
+    if not t:
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail="Le titre est requis.",
+        )
+    if len(t) > ARTICLE_TITLE_MAX_LEN_VINTED:
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail=(
+                f"Le titre ne doit pas dépasser {ARTICLE_TITLE_MAX_LEN_VINTED} caractères "
+                f"(limite Vinted, espaces inclus). Longueur : {len(t)}."
+            ),
+        )
+    return t
+
+
 def _validate_graded_article_or_raise(article: Article) -> None:
     if not article.is_graded:
         return
@@ -472,9 +497,11 @@ async def create_article(
         grade_vid = None
         cert_norm = None
 
+    title_clean = _validate_create_title_or_raise(title)
+
     article = Article(
         user_id=user.id,
-        title=title.strip(),
+        title=title_clean,
         description=description,
         pokemon_name=pokemon_name.strip() if pokemon_name else None,
         set_code=set_code.strip() if set_code else None,

api/schemas/articles.py

@@ -1,7 +1,10 @@
 from decimal import Decimal
 from typing import Literal
 
-from pydantic import BaseModel, Field
+from pydantic import BaseModel, Field, field_validator
+
+# Limite champ titre Vinted (espaces comptés comme un caractère).
+ARTICLE_TITLE_MAX_LEN_VINTED = 100
 
 
 class ArticleUpdate(BaseModel):
@@ -17,6 +20,23 @@ class ArticleUpdate(BaseModel):
     graded_cert_number: str | None = None
     purchase_price: Decimal | None = None
     sell_price: Decimal | None = None
+    #: Remet l’article en « non publié » côté GoupixDex (ne supprime pas l’annonce sur Vinted).
+    clear_vinted_publication: bool | None = None
+    #: Idem eBay : efface le suivi local et l’identifiant d’annonce enregistré.
+    clear_ebay_publication: bool | None = None
+
+    @field_validator("title")
+    @classmethod
+    def title_vinted_max_length(cls, v: str | None) -> str | None:
+        if v is None:
+            return None
+        s = v.strip()
+        if len(s) > ARTICLE_TITLE_MAX_LEN_VINTED:
+            raise ValueError(
+                f"Le titre ne doit pas dépasser {ARTICLE_TITLE_MAX_LEN_VINTED} caractères "
+                f"(limite Vinted, espaces inclus). Longueur : {len(s)}."
+            )
+        return s
 
 
 class SoldPatch(BaseModel):

api/services/article_service.py

@@ -148,3 +148,10 @@ def update_article_from_body(article: Article, body: ArticleUpdate) -> None:
         else:
             c = str(cert).strip()[:30]
             article.graded_cert_number = c or None
+    if data.get("clear_vinted_publication") is True:
+        article.published_on_vinted = False
+        article.vinted_published_at = None
+    if data.get("clear_ebay_publication") is True:
+        article.published_on_ebay = False
+        article.ebay_listing_id = None
+        article.ebay_published_at = None

web/app/components/articles/ArticleForm.vue

@@ -27,6 +27,9 @@ const emit = defineEmits<{
   submitEdit: [body: ArticleUpdateBody]
 }>()
 
+/** Limite titre Vinted : espaces comptés comme un caractère. */
+const VINTED_TITLE_MAX_CHARS = 100
+
 const title = ref('')
 const description = ref('')
 const pokemonName = ref('')
@@ -45,6 +48,9 @@ const isGraded = ref(false)
 const gradedGraderValueId = ref('')
 const gradedGradeValueId = ref('')
 const gradedCertNumber = ref('')
+/** Édition : cocher pour enregistrer l’article comme non publié sur le canal (suivi GoupixDex uniquement). */
+const clearVintedPublication = ref(false)
+const clearEbayPublication = ref(false)
 const purchasePrice = ref('')
 const sellPrice = ref('')
 const toast = useToast()
@@ -83,6 +89,23 @@ const canPublishEbay = computed(
     && svcSettings.value?.ebay_listing_config_complete === true
 )
 
+const titleLenVinted = computed(() => title.value.trim().length)
+
+function titleWithinVintedLimit(): boolean {
+  return titleLenVinted.value <= VINTED_TITLE_MAX_CHARS
+}
+
+function assignTitleFromExternal(raw: string) {
+  title.value = raw.trim().slice(0, VINTED_TITLE_MAX_CHARS)
+}
+
+const titleFieldDescription = computed(() => {
+  const n = titleLenVinted.value
+  const max = VINTED_TITLE_MAX_CHARS
+  const base = `${n} / ${max} caractères (limite Vinted, espaces inclus)`
+  return n > max ? `${base} — raccourcissez avant enregistrement` : base
+})
+
 onMounted(async () => {
   try {
     svcSettings.value = await getSettings()
@@ -139,6 +162,8 @@ watch(
         ? a.graded_grade_value_id
         : ''
     gradedCertNumber.value = a.graded_cert_number ?? ''
+    clearVintedPublication.value = false
+    clearEbayPublication.value = false
   },
   { immediate: true }
 )
@@ -175,7 +200,7 @@ function applyScanPrefill(scan: {
   ocr: Record<string, unknown>
   pricing: { cardmarket_eur: number | null, tcgplayer_usd: number | null }
 }) {
-  title.value = scan.listing_preview.title
+  assignTitleFromExternal(scan.listing_preview.title)
   description.value = scan.listing_preview.description
   const o = scan.ocr
   const en = typeof o.pokemon_name_english === 'string' ? o.pokemon_name_english : ''
@@ -214,7 +239,7 @@ async function applyEbayPrefill(p: {
   imageUrl?: string | null
 }) {
   if (p.title) {
-    title.value = p.title
+    assignTitleFromExternal(p.title)
   }
   if (p.description) {
     description.value = p.description
@@ -262,7 +287,7 @@ async function applyWardrobeSlot(p: WardrobeSlotPrefill) {
   wardrobeVintedListed.value = p.wardrobeVintedListed
   wardrobeVintedPublishedAtIso.value = p.vintedPublishedAtIso
   wardrobeImportSoldPrice.value = p.importSoldPrice
-  title.value = p.title
+  assignTitleFromExternal(p.title)
   description.value = p.description
   purchasePrice.value = p.purchasePrice || '0'
   sellPrice.value = p.sellPrice
@@ -306,6 +331,9 @@ defineExpose({
 })
 
 function buildCreateFormData(): FormData {
+  if (!titleWithinVintedLimit()) {
+    throw new Error('ARTICLE_TITLE_TOO_LONG')
+  }
   const fd = new FormData()
   fd.append('title', title.value.trim())
   fd.append('description', description.value)
@@ -369,6 +397,22 @@ function imageSrc(url: string) {
 }
 
 function submit() {
+  if (!title.value.trim()) {
+    toast.add({
+      title: 'Titre requis',
+      description: 'Indiquez un titre pour l’article.',
+      color: 'error'
+    })
+    return
+  }
+  if (!titleWithinVintedLimit()) {
+    toast.add({
+      title: 'Titre trop long',
+      description: `Vinted : maximum ${VINTED_TITLE_MAX_CHARS} caractères (espaces inclus). Actuellement : ${title.value.trim().length}.`,
+      color: 'error'
+    })
+    return
+  }
   if (isGraded.value) {
     if (!gradedGraderValueId.value || !gradedGradeValueId.value) {
       toast.add({
@@ -383,7 +427,7 @@ function submit() {
     emit('submitCreate', buildCreateFormData())
     return
   }
-  emit('submitEdit', {
+  const editBody: ArticleUpdateBody = {
     title: title.value.trim(),
     description: description.value,
     pokemon_name: pokemonName.value.trim() || null,
@@ -400,15 +444,30 @@ function submit() {
     sell_price: sellPrice.value.trim()
       ? Number(sellPrice.value.replace(',', '.'))
       : null
-  })
+  }
+  if (clearVintedPublication.value) {
+    editBody.clear_vinted_publication = true
+  }
+  if (clearEbayPublication.value) {
+    editBody.clear_ebay_publication = true
+  }
+  emit('submitEdit', editBody)
 }
 </script>
 
 <template>
   <div class="space-y-6">
     <div class="grid gap-4 sm:grid-cols-2">
-      <UFormField label="Titre" required>
-        <UInput v-model="title" class="w-full" />
+      <UFormField
+        label="Titre"
+        required
+        :description="titleFieldDescription"
+      >
+        <UInput
+          v-model="title"
+          class="w-full"
+          :maxlength="mode === 'create' ? VINTED_TITLE_MAX_CHARS : undefined"
+        />
       </UFormField>
       <UFormField v-if="!isGraded" label="État">
         <USelect
@@ -628,6 +687,34 @@ function submit() {
       </div>
     </div>
 
+    <div
+      v-if="
+        mode === 'edit'
+          && article
+          && !article.is_sold
+          && ((article.published_on_vinted ?? false) || (article.published_on_ebay ?? false))
+      "
+      class="rounded-lg border border-default p-4 space-y-3"
+    >
+      <p class="text-sm font-medium text-highlighted">
+        Statut de publication dans GoupixDex
+      </p>
+      <p class="text-xs text-muted leading-relaxed">
+        Cochez pour indiquer que l’article n’est plus « en ligne » dans l’app. Cela ne retire pas l’annonce sur Vinted ou eBay :
+        supprimez-la sur le site si besoin. Utile pour relancer une publication depuis GoupixDex.
+      </p>
+      <UCheckbox
+        v-if="article.published_on_vinted ?? false"
+        v-model="clearVintedPublication"
+        label="Ne plus marquer comme publié sur Vinted"
+      />
+      <UCheckbox
+        v-if="article.published_on_ebay ?? false"
+        v-model="clearEbayPublication"
+        label="Ne plus marquer comme publié sur eBay"
+      />
+    </div>
+
     <UButton
       v-if="showSubmitButton"
       color="primary"

web/app/composables/useArticles.ts

@@ -102,6 +102,10 @@ export interface ArticleUpdateBody {
   graded_cert_number?: string | null
   purchase_price?: number
   sell_price?: number | null
+  /** Édition : remet le suivi Vinted à « non publié » dans GoupixDex. */
+  clear_vinted_publication?: boolean
+  /** Édition : remet le suivi eBay à « non publié » dans GoupixDex. */
+  clear_ebay_publication?: boolean
 }
 
 export function useArticles() {

web/app/pages/articles/[id].vue

@@ -56,6 +56,7 @@ async function onSubmitEdit(body: ArticleUpdateBody) {
   submitting.value = true
   try {
     const updated = await updateArticle(id.value, body)
+    article.value = updated
     toast.add({ title: 'Article mis à jour', color: 'success' })
     const listed =
       Boolean(updated.published_on_vinted ?? false) || Boolean(updated.published_on_ebay ?? false)

web/app/pages/articles/batch-create.vue

@@ -250,7 +250,20 @@ async function submitAll() {
         })
         return
       }
-      const fd = comp.buildCreateFormData()
+      let fd: FormData
+      try {
+        fd = comp.buildCreateFormData()
+      } catch (e) {
+        if (e instanceof Error && e.message === 'ARTICLE_TITLE_TOO_LONG') {
+          toast.add({
+            title: `Article ${i + 1} — titre trop long`,
+            description: 'Maximum 100 caractères pour Vinted (espaces inclus). Raccourcissez le titre.',
+            color: 'error'
+          })
+          return
+        }
+        throw e
+      }
       const title = fd.get('title')?.toString()?.trim()
       const purchase = fd.get('purchase_price')?.toString()?.trim()
       const images = fd.getAll('images') as File[]