Shopify-themareviewProgrammeertraining

Codereview van een Shopify Liquid-thema

Meestal klaar in 15 minutenRevisies inbegrepenKwaliteitscontrole
Pakketten
BetaalbeschermingDetails
  • Revisies binnen het pakket zijn gratis.
  • Reageer je niet binnen 3 dagen na levering, dan wordt het werk automatisch goedgekeurd.

Het geld wordt bij betaling gereserveerd en pas vrijgegeven als je het werk goedkeurt.

Codereview van een Shopify Liquid-thema

Voorbeeldresultaat

Voorbeeld voor het pakket Standaard

Code review: Python order-import script for a small online shop

What the code does

  1. Reads orders.csv from the shop export.
  2. For each order line, finds the product by SKU in stock.xlsx and subtracts the quantity.
  3. Writes the updated stock back and logs totals to import_log.txt.

The structure is clear and functions are well named. Most issues are about input that does not look the way the script expects.

Bugs and weak spots

#LineSeverityIssue
148HighThe same CSV can be imported twice; there is no check for already processed order IDs. This is the most likely cause of negative stock.
272Highint(row["qty"]) crashes on empty or "2.0" values from refunded orders.
395MediumSKU lookup is case-sensitive: "gt-101" and "GT-101" are treated as different products.
4130MediumThe file is opened without encoding="utf-8", so product names with accents break on Windows.
5201LowBare except: hides real errors and makes the log useless.

Corrected fragments

Issue 1: skip orders already imported

processed = load_processed_ids("processed_orders.txt")
for row in reader:
    if row["order_id"] in processed:
        continue
    # ... existing logic ...
    processed.add(row["order_id"])
save_processed_ids("processed_orders.txt", processed)

Why: each order is applied once, even if you run the script on the same file again.

Issue 2: read quantity safely

raw = (row.get("qty") or "").strip()
try:
    qty = int(float(raw))
except ValueError:
    log.warning("Order %s: bad quantity %r, skipped", row["order_id"], raw)
    continue

Why: refunded or partial lines are skipped with a clear note instead of stopping the whole import.

Issue 3: normalise SKUs

sku = row["sku"].strip().upper()

Apply the same .strip().upper() when building the stock dictionary.

Style and readability

  • process() is 90 lines long; split it into read_orders, apply_to_stock, write_report.
  • Replace magic column numbers (row[7]) with header names.

Please test each fix on a copy of your files: we read the code but did not run it.

Client: owner of a tea shop who wrote a 310-line Python script that imports orders from a CSV export into the stock spreadsheet Problem reported: "Sometimes stock goes negative, and the script crashes on some files."

De klant en alle details zijn verzonnen.

Over deze dienst

Wijzigingen aan Shopify-thema's gaan vaak mis in de Liquid zelf: een sectieschema met een kapotte instelling, een lus over een hele collectie op elke pagina, de oude include-tag in plaats van render, of een snippet van een forum die botst met de eigen scripts van het thema. Plak de code van de sectie, snippet of template en wij leggen uit wat die uitvoert, waarom het de winkel of de thema-editor breekt en hoe je het oplost. Gecorrigeerde fragmenten volgen de conventies van Online Store 2.0, zodat ze blijven werken in de customizer.

Wat is inbegrepen

  • Wat je krijgtEen schriftelijke review op de bestelpagina: wat je Liquid uitvoert, problemen op belang en gecorrigeerde fragmenten van secties, snippets en schema's. Premium voegt een plan toe om vastgecodeerde aanpassingen te verplaatsen naar thema-instellingen die de winkeleigenaar kan wijzigen.
  • Liquid-logicaObjecten, filters en voorwaarden
  • SectieschemaInstellingen en blokken die in de editor laden
  • render in plaats van includeActuele tags en snippetvariabelen
  • Zware lussenMinder objecten geladen per pagina
  • Gecorrigeerde fragmentenKlaar om in de code-editor te plakken

Hoe het werkt

  1. 1
    BriefingJe plakt de Liquid en een screenshot
  2. 2
    LezenWij volgen wat de template uitvoert
  3. 3
    ReviewJe ontvangt problemen en gecorrigeerde code

Wat we van je nodig hebben

De Liquid-code van de sectie, snippet of template als geplakte tekst of screenshots, de naam en versie van het thema, wat er in de winkel moet verschijnen en een screenshot van het probleem.

We vragen je:

  • Welk thema en welke versie gebruik je?
  • Om welk bestand gaat het?
  • Wat moet er in de winkel verschijnen en wat verschijnt er in plaats daarvan?
  • Begon het probleem na het installeren van een app of een thema-update?

Pakketten vergelijken

PakkettenBasisLiquid tot 100 regels€ 14StandaardThemacode tot 400 regels€ 32PremiumTot 1.000 regels met een instellingenplan€ 62
Levertijd1 dag2 dagen3 dagen
Revisies123
Regels codetot 100tot 400tot 1.000
Gecorrigeerde fragmentenJaJaJa
Plan voor thema-instellingenNeeNeeJa
Bestellen BasisBestellen StandaardBestellen Premium

Betaalbescherming

Je betaalt bij het bestellen, maar het geld wordt alleen vrijgegeven voor goedgekeurd werk.

  • Vaste prijsDe prijs van het pakket verandert niet meer nadat je hebt betaald.
  • Het geld wordt gereserveerdBij betaling wordt het bedrag alleen gereserveerd. Het gaat pas naar de dienst als je het werk goedkeurt.
  • Gecontroleerd vóór leveringElk resultaat doorloopt een kwaliteitscontrole voordat het bij jou komt, en je ziet het op de bestelpagina voordat je het goedkeurt.
  • Gratis revisiesVraag wijzigingen binnen je briefing aan, zo vaak als het pakket toestaat.
  • Goedkeuring na 3 dagenKeur je het werk binnen 3 dagen na levering niet goed en vraag je ook geen wijzigingen, dan wordt het automatisch goedgekeurd. Schrijf je na levering in de chat van de bestelling, dan wacht de bestelling; zodra we hebben geantwoord, kunnen we de 3 dagen opnieuw laten ingaan en melden we dat in de chat.
  • Opnieuw doen of volledige terugbetalingPast het resultaat niet bij je briefing en hebben de revisies dat niet opgelost, of duurt het werk meer dan twee keer zo lang als de levertijd, schrijf dan in de chat van de bestelling: we doen het werk opnieuw of betalen het volledige bedrag terug.
Hoe het werkt →

Vragen en antwoorden

Kunnen jullie mijn winkel rechtstreeks bewerken?

Nee, wij hebben geen toegang als medewerker nodig. Dupliceer je thema, plak onze oplossingen in de kopie en bekijk een voorbeeld voordat je publiceert.

Voeren jullie mijn code uit?

Nee. Wij lezen hem zorgvuldig en redeneren over hoe hij zich gedraagt, maar voeren hem niet uit, dus test elke voorgestelde oplossing in je eigen project.

Verhoogt dit de snelheidsscore van mijn winkel?

Wij halen zware Liquid weg uit de code die je stuurt, maar de score hangt ook af van apps, afbeeldingen en scripts die wij niet zien.

Staat je antwoord er niet bij?

Pakketten