Some doc stuff

Also delete compiled entry JS and add a Makefile

Dockerfile

@@ -1,25 +0,0 @@
-FROM debian:latest as builder
-
-RUN apt-get update 
-RUN apt-get upgrade -y
-RUN apt-get install -y elm-compiler make ca-certificates
-
-COPY . /app
-WORKDIR /app
-RUN rm -rf .venv venv
-
-RUN make frontend
-
-
-FROM python:3.11-alpine as runner
-COPY --from=builder /app /app
-
-WORKDIR /app
-RUN pip install -r requirements.txt
-RUN pip install gunicorn
-
-EXPOSE 5000
-ENV JON_DB_CONNECTION_STRING="host=fsmi-db.fsmi.org dbname=garfield"
-ENV JON_SECRET_KEY="changeme"
-
-CMD ["sh", "-c", "gunicorn -b '0.0.0.0:5000' --chdir /app 'jon:create_app()'"]

README.md

@@ -4,11 +4,6 @@
 
 ## Setup
 
-`jon` is a Python WSGI application written using Flask.
-This means you'll have to install a bunch of Python packages to get up and running.
-
-### Dependencies
-
 ```
 pip install -r requirements.txt
 ```
@@ -16,14 +11,6 @@ pip install -r requirements.txt
 You should probably use a virtualenv for that.
 I develop `jon` using Python 3.10 but it should work with older versions as well.
 
-#### Arch Linux
-
-If you're on Arch, these packages are required for running the server:
-
-```
-python python-flask python-flask-login python-psycopg2
-```
-
 ### Building Frontend JS
 
 Most of jon works without JS but there are some features that require it.
@@ -45,20 +32,6 @@ flask --app jon run --debug
 
 `--debug` restarts the server when a source file changes.
 
-## Running with docker
-When you prefer running the application using docker you can just use
-```
-docker compose up
-```
-
-In case your local username does not line up with your FSMI-username, you need to specify your FSMI-username
-using `USER=<username>`, e.g.:
-```
-USER=shirkanesi docker compose up
-```
-
-This can also be persisted by following the instructions in the docker-compose.yml
-
 ## fsmi-db forward
 
 ```
@@ -73,13 +46,9 @@ ssh -nNTvL 5432:fsmi-db.fsmi.org:5432 fsmi-login.fsmi.uni-karlsruhe.de
   - [ ] etc.
   - [ ] Make it print nicely
 - [ ] Make it possible to edit entries
-- [ ] Fix unsafe client-side sessions, either:
-  - [ ] Use `flask-session` for file-backed sessions
-  - [ ] Use `flask-login` with a single user stored in memory
 - [ ] Improve project structure
 - [ ] Use `flask.flash` for error messages
 - [x] Implement item and snack entry as Elm application
   - [x] Figure out/Add documentation about building `entry.js`
   - [ ] Clean up the code a little and add some comments
   - [ ] Needs good documentation for maintainability
-- [ ] Use cool new function for deactivating items

docker-compose.yml

@@ -1,19 +0,0 @@
----
-version: '3.7'
-
-services:
-  jon:
-    container_name: jon
-    build: .
-    ports:
-      - "5000:5000"
-    volumes:
-      - ~/.pgpass:/root/.pgpass:ro
-    dns:
-      - 1.1.1.1
-      - 8.8.8.8
-    network_mode: bridge
-    environment:
-      # If your local user is different from your fsmi user, change $USER here!
-      - JON_DB_CONNECTION_STRING="host=fsmi-db.fsmi.org dbname=garfield user=$USER"
-      - JON_SECRET_KEY="changemetosomethingsuperrandomandsecure"

frontend/Calculator.elm

@@ -9,15 +9,7 @@ import Select
 
 type Tax = Net | Gross
 
--- Duplicated from Entry.elm but too lazy to sandwich this out
-type alias TaxGroup =
-  { id : Int
-  , description : String
-  , percentage : Float
-  }
-
-showTax : Tax -> String
-showTax tax = case tax of
+ctShow ct = case ct of
   Gross -> "Brutto"
   Net -> "Netto"
 
@@ -27,13 +19,11 @@ type alias Model =
   , bundleSize : NumberInput.Model Int
   }
 
-init : Float -> Model
 init bundlePrice = Model
-  (Select.init showTax showTax Net [Net, Gross])
+  (Select.init ctShow ctShow Net [Net, Gross])
   (NumberInput.fromFloat bundlePrice)
   (NumberInput.fromInt 1)
 
-getResult : Model -> TaxGroup -> Maybe Float
 getResult model taxGroup =
   case (NumberInput.get model.bundlePrice, NumberInput.get model.bundleSize) of
     (Just bundlePrice, Just bundleSize) ->
@@ -51,7 +41,6 @@ type Msg
   | SetBundlePrice String
   | SetBundleSize String
 
-update : Msg -> Model -> Model
 update msg model = case msg of
   SetTax key ->
     { model | tax = Select.update key model.tax }
@@ -60,7 +49,6 @@ update msg model = case msg of
   SetBundleSize str ->
     { model | bundleSize = NumberInput.update str model.bundleSize }
 
-view : Model -> TaxGroup -> Html Msg
 view model taxGroup =
   let
     mainPart =
@@ -124,5 +112,4 @@ view model taxGroup =
         ]
       ]
 
-roundTo : Int -> Float -> Float
-roundTo places x = toFloat (round <| x * 10 ^ toFloat places) / 10 ^ toFloat places
+roundTo places x = toFloat (round <| x * 10 ^ places) / 10 ^ places

frontend/Entry.elm

@@ -14,50 +14,6 @@ import Calculator
 import NumberInput
 import Select
 
-{-
-Elm forces us to use the Elm architecture:
-
-              ┌──────┐
- ┌─────Model──► view ├──Html───────┐
- │            └──────┘             │
- │                                 │
-┌┴─────────────────────────────────▼┐
-│            Elm runtime            │
-└▲─────────────────────────────────┬┘
- │                                 │
- │            ┌──────┐             │
- └─Model+Cmd──┤update◄──Msg+Model──┘
-              └──────┘
-
-This architecture is similar to what React does but its implementation
-in Elm is a bit special since it's purely functional and side effects
-are isolated into the runtime system.
-
-An Elm component is usually centered around two types, Model and Msg.
-Model contains all data the application is concerned with, including the state
-of UI elements. Msg encodes all updates to Model that the application supports.
-
-In addition to Msg and Model, we have to provide two functions, view and update.
-
-view : Model -> Html Msg
-update : Msg -> Model -> (Model, Cmd Msg)
-
-view maps a Model to a DOM tree. Events in this DOM tree create Msg values.
-update maps a Msg and a Model to a new Model. In addition, update can create
-a command. Commands are used to make the runtime do side effects, which in
-turn create new Msg values.
-
-For example, we have a SetSearchTerm message which simply updates the searchTerm
-property in the model. This message is triggered every time the search box input
-is changed. Submitting the search box form triggers a SubmitSearch event.
-This event leaves the model unchanged but issues a command that sends the search
-term to a JSON endpoint. When the request successfully resolves, the runtime
-triggers a ReceiveSearchResults messages which updates the list of search results
-in the model.
-
-See Calculator.elm for a simpler example of this architecture.
--}
-
 main = Browser.element
   { init = \globals ->
     ( Context globals <| ItemSearch { searchTerm = "", searchResults = [] }
@@ -133,11 +89,8 @@ type alias Context =
 
 type alias Globals =
   { locations : List Location
-  , defaultLocation : Location
   , groups : List Group
-  , defaultGroup : Group
   , taxGroups : List TaxGroup
-  , defaultTaxGroup : TaxGroup
   }
 
 type State
@@ -161,7 +114,7 @@ type Msg
   = SetSearchTerm String
   | SubmitSearch
   | ReceiveSearchResults (Result Http.Error (List SearchResult))
-  | GotoItemEditor IEInit
+  | GotoItemEditor SearchResult
   | SetBarcode String
   | SetName String
   | SetSalesUnits String
@@ -172,10 +125,6 @@ type Msg
   | SetLocation String
   | SetTaxGroup String
 
-type IEInit
-  = IEInitBarcode String
-  | IEInitSearchResult SearchResult
-
 -- Update logic: State machine etc.
 
 update msg { globals, state } =
@@ -197,21 +146,7 @@ updateState msg globals state = case state of
       )
     ReceiveSearchResults (Ok searchResults) ->
       (ItemSearch { model | searchResults = searchResults }, Cmd.none)
-    GotoItemEditor (IEInitBarcode barcode) ->
-      ( ItemEditor
-        { barcode = barcode
-        , name = ""
-        , calculator = Calculator.init 0
-        , netUnitPrice = NumberInput.fromFloat 0
-        , grossUnitPrice = NumberInput.fromFloat 0
-        , salesUnits = NumberInput.fromInt 0
-        , group = Select.init (.id >> String.fromInt) (.name) globals.defaultGroup globals.groups
-        , location = Select.init (.id >> String.fromInt) (.name) globals.defaultLocation globals.locations
-        , taxGroup = Select.init (.id >> String.fromInt) (.description) globals.defaultTaxGroup globals.taxGroups
-        }
-      , Cmd.none
-      )
-    GotoItemEditor (IEInitSearchResult searchResult) ->
+    GotoItemEditor searchResult ->
       case find (\tg -> tg.id == searchResult.taxGroupId) globals.taxGroups of
         Nothing -> (state, Cmd.none)
         Just taxGroup ->
@@ -266,21 +201,14 @@ suggestedGrossPrice netPrice percentage =
 
 view { globals, state } = case state of
   ItemSearch model ->
-    div []
-      [ div []
-        [ if model.searchTerm == ""
-          then button [ disabled True ] [ text "Neuer Artikel" ]
-          else button [ onClick <| GotoItemEditor <| IEInitBarcode model.searchTerm  ] [ text <| "Neuer Artikel mit Barcode " ++ model.searchTerm ]
-        ]
-      , fieldset []
-        [ legend [] [ text "Vorlage für Auftrag wählen" ]
-        , Html.form [ onSubmit SubmitSearch ]
-          [ div [ class "form-input" ]
-            [ label [ for "search-term", title "Barcode oder Name" ] [ text "Suchbegriff" ]
-            , input [ onInput SetSearchTerm, value model.searchTerm, id "search-term" ] []
-            ]
-          , table [] <| searchResultHeaders :: List.map viewSearchResult model.searchResults
+    fieldset []
+      [ legend [] [ text "Vorlage für Auftrag wählen" ]
+      , Html.form [ onSubmit SubmitSearch ]
+        [ div [ class "form-input" ]
+          [ label [ for "search-term", title "Barcode oder Name" ] [ text "Suchbegriff" ]
+          , input [ onInput SetSearchTerm, value model.searchTerm, id "search-term" ] []
           ]
+        , table [] <| searchResultHeaders :: List.map viewSearchResult model.searchResults
         ]
       ]
   ItemEditor model ->
@@ -413,7 +341,7 @@ viewSearchResult model =
     , td [] [ text model.locationName ]
     , td [] [ text <| showBool model.available ]
     , td []
-      [ Html.form [ onSubmit <| GotoItemEditor <| IEInitSearchResult model ]
+      [ Html.form [ onSubmit <| GotoItemEditor model ]
         [ button [] [ text "Als Vorlage verwenden" ]
         ]
       ]

jon/__init__.py

@@ -20,10 +20,14 @@ def create_app():
     # You don't need a config.json. If you don't provide one, default-config.json
     # is used.
     app.config.from_file("config.json", load=json.load, silent=True)
-    app.config.from_prefixed_env(prefix="JON")
 
     db.init_app(app)
-    auth.init_app(app)
+
+    # This function denies every request until `auth.ACCESS_TOKEN`
+    # is passed using `?token=` to authenticate the session.
+    @app.before_request
+    def before_req_fun():
+        return auth.before_request()
 
     @app.context_processor
     def utility_processor():

jon/auth.py

@@ -1,10 +1,7 @@
 import secrets
 import string
 
-from flask import Blueprint, request, redirect, render_template
-from flask_login import current_user, login_user, logout_user, LoginManager
-from typing import Any, Dict, List, Optional
-
+from flask import Blueprint, request, redirect, render_template, session
 
 bp = Blueprint("auth", __name__, url_prefix="/auth")
 
@@ -18,67 +15,27 @@ ALLOWED_PATHS = [
 ]
 
 
-# A poor man's replacement for memory-backed session solution.
-# We keep exactly one User (and the corresponding UserData) in
-# memory and use that to store session data.
-class UserData:
-    location: Optional[Dict[str, Any]]
-    orders: List[Dict[str, Any]]
+def before_request():
+    """
+    If the correct token query parameter is passed along with any request,
+    we mark this session authenticated by setting `session["authenticated"]`.
+    Unless the session is authenticated, all requests result in a 403 FORBIDDEN.
+    """
+    if "token" in request.args:
+        if request.args["token"] == ACCESS_TOKEN:
+            session["authenticated"] = ()
+        # Reload the page without query parameters
+        return redirect(request.path)
 
-    def __init__(self):
-        self.location = None
-        self.orders = []
+    # Don't deny any paths in `ALLOWED_PATHS`
+    if request.path in ALLOWED_PATHS:
+        return
 
-
-class User:
-    is_authenticated: bool
-    is_active: bool
-    is_anonymous: bool
-
-    data: UserData
-
-    def __init__(self):
-        self.is_authenticated = True
-        self.is_active = True
-        self.is_anonymous = False
-
-        self.data = UserData()
-
-    def get_id(self) -> str:
-        return ""
-
-
-def init_app(app):
-    login_manager = LoginManager(app)
-    the_one_and_only_user = User()
-
-    @login_manager.user_loader
-    def load_user(user_id: str) -> User:
-        assert user_id == ""
-        return the_one_and_only_user
-
-    # This function denies every request until `auth.ACCESS_TOKEN`
-    # is passed using `?token=` to authenticate the user.
-    # We use this instead of @login_required because otherwise we'd have
-    # to add that annotation to all routes.
-    # See also: https://flask-login.readthedocs.io/en/latest/#flask_login.login_required
-    @app.before_request
-    def before_request():
-        if "token" in request.args:
-            if request.args["token"] == ACCESS_TOKEN:
-                login_user(the_one_and_only_user)
-            # Reload the page without query parameters
-            return redirect(request.path)
-
-        # Never deny any paths in `ALLOWED_PATHS`
-        if request.path in ALLOWED_PATHS:
-            return
-
-        if not current_user.is_authenticated:
-            return render_template("auth/denied.html"), 403
+    if not "authenticated" in session:
+        return render_template("auth/denied.html"), 403
 
 
 @bp.get("/logout")
 def logout():
-    logout_user()
+    session.pop("authenticated", None)
     return redirect("/")

jon/db/add_views.sql

@@ -58,56 +58,3 @@ FROM garfield.inventory_items
   ) m USING (item_id)
 ORDER BY inventory_items.name;
 
--- How many *other* active inventory lines exist with the same barcode in the same location that are newer?
-CREATE TEMPORARY VIEW more_recent_inventory_lines_with_same_barcode AS
-SELECT
-  a.item_id,
-  -- It's important not to count(*) here because item_id is NULL
-  -- when no other inventory lines exist.
-  count(b.item_id) AS other_lines_count
-FROM garfield.inventory_items AS a
-LEFT JOIN garfield.inventory_items AS b
-       ON a.item_barcode = b.item_barcode
-      AND a.item_id != b.item_id
-      AND a.location = b.location
-      AND b.available
-      AND b.bought > a.bought
-GROUP BY a.item_id;
-
--- We need to create this table so that we can put an index on it
--- Otherwise the join in the consumption graph query becomes much slower
--- Perhaps it would be nicer to use a materialized view instead
-DROP TABLE IF EXISTS last_n_days;
-CREATE TEMPORARY TABLE last_n_days AS (
-  SELECT
-    generate_series(now() - interval '14 days', now(), interval '1 day')::date AS sale_date
-);
-CREATE UNIQUE INDEX last_n_days_sale_date ON last_n_days (sale_date);
-
--- Get an array of how often items were sold over the last 14 days
-CREATE TEMPORARY VIEW inventory_last_n_days_sales AS
-WITH
-sales_by_date AS (
-  SELECT
-    inventory_line AS item_id,
-    date_trunc('day', snack_sales_log_timestamp AT TIME ZONE 'UTC+1') AS sale_date,
-    count(*)::int AS sales
-  FROM garfield.snack_sales_log
-  GROUP BY item_id, sale_date
-),
-beeg AS (
-  SELECT item_id, sale_date, count(snack_sales_log_timestamp)::int AS sales
-  FROM garfield.inventory_items
-  CROSS JOIN last_n_days
-  LEFT JOIN garfield.snack_sales_log
-         ON inventory_line = item_id
--- snack_sales_log has an index on snack_sales_log_timestamp to speed up this query
--- If that index doesn't exist the query takes much longer.
-        AND sale_date = date_trunc('day', snack_sales_log_timestamp AT TIME ZONE 'UTC+1')
-  WHERE available
-  GROUP BY item_id, sale_date
-  ORDER BY item_id, sale_date
-)
-SELECT item_id, array_agg(sales) AS last_n_days_sales
-FROM beeg
-GROUP BY item_id

jon/db/get_inventory_overview.sql

@@ -1,8 +1,6 @@
 SELECT
   *
 FROM all_inventory_item_overview
-LEFT JOIN more_recent_inventory_lines_with_same_barcode USING (item_id)
-LEFT JOIN inventory_last_n_days_sales USING (item_id)
 WHERE (%(location_id)s IS NULL OR location = %(location_id)s)
   AND available
 ORDER BY

jon/db/get_inventory_report.sql

@@ -1,43 +1,49 @@
--- parameters:
---
--- location_id: Location to generate the report for
-
 WITH
-sales_by_item_id AS (
-  SELECT
-    inventory_line AS item_id,
-    max(snack_sales_log_timestamp) AS last_sold,
-    count(*)::int AS units_sold
+most_recent_sales AS (
+  SELECT DISTINCT ON (inventory_line)
+    inventory_line, snack_sales_log_id, snack_sales_log_timestamp AS most_recent_sale
   FROM garfield.snack_sales_log
-  WHERE type_id = 'SNACK_BUY'
-    AND inventory_line IS NOT NULL
-    AND snack_sales_log_timestamp > NOW() - INTERVAL '120 days'
-    AND location_id = %(location_id)s
-  GROUP BY item_id
+  ORDER BY inventory_line ASC, snack_sales_log_timestamp DESC
 ),
-sales_by_barcode AS (
+enhanced_overview1 AS (
   SELECT
-    item_barcode,
-    max(name) AS name,
-    sum(units_sold) AS units_sold,
-    round(avg((units_sold / extract(epoch FROM (CASE WHEN available THEN now() ELSE last_sold END) - bought)) * 86400)::numeric, 2) AS sold_per_day
-  FROM sales_by_item_id
+    inventory_items.item_id,
+    inventory_items.item_barcode,
+    inventory_items.name,
+    units_left,
+    inventory_items.sales_units,
+    correction_delta,
+    location_name,
+    location,
+      CASE
+        WHEN snack_sales_log_id IS NULL THEN 0
+        ELSE sales / (EXTRACT(EPOCH FROM (
+          CASE
+            WHEN units_left <= 0 THEN most_recent_sale
+            ELSE NOW()
+          END
+        )) - EXTRACT(EPOCH FROM bought)) * 24 * 3600
+    END AS per_day
+  FROM garfield.inventory_item_overview
   LEFT JOIN garfield.inventory_items USING (item_id)
-  GROUP BY item_barcode
+  LEFT JOIN most_recent_sales ON item_id = inventory_line
 ),
-current_inventory AS (
+enhanced_overview2 AS (
   SELECT
-    item_barcode,
-    sum(units_left)::int AS units_left
-  FROM all_inventory_item_overview
-  WHERE available
-    AND location = %(location_id)s
-  GROUP BY item_barcode
+    *,
+    CASE
+      WHEN per_day = 0 THEN NULL
+      ELSE GREATEST(0, units_left / per_day)
+    END AS days_left
+  FROM enhanced_overview1
 )
 
 SELECT
-  sales_by_barcode.*,
-  COALESCE(current_inventory.units_left, 0)::int AS units_left
-FROM sales_by_barcode
-LEFT JOIN current_inventory USING (item_barcode)
-ORDER BY units_sold DESC
+  *,
+  CASE
+    WHEN days_left IS NULL THEN NULL
+    ELSE GREATEST(0, (60 - days_left) * per_day)
+  END AS for_two_months
+FROM enhanced_overview2
+WHERE (%(location_id)s IS NULL OR location = %(location_id)s)
+ORDER BY days_left ASC, per_day DESC

jon/entry.py

@@ -1,5 +1,5 @@
-from flask import Blueprint, flash, redirect, render_template, request
-from flask_login import current_user
+from flask import Blueprint, flash, redirect, render_template, request, session
+
 
 from . import db
 
@@ -9,18 +9,22 @@ bp = Blueprint("entry", __name__, url_prefix="/entry")
 
 @bp.route("/", methods=["GET", "POST"])
 def index():
+    cart = session.get("cart", default=[])
     return render_template(
-        "entry/index.html"
+        "entry/index.html",
+        cart=cart
     )
 
 
 @bp.post("/add-new-items")
 def add_new_entries():
+    print(session)
+
     i_know_what_im_doing = "i-know-what-im-doing" in request.form
     if not i_know_what_im_doing:
         return "Du weißt nicht was du tust", 400
 
-    orders = current_user.data.orders
+    orders = session.get("cart", default=[])
     if not orders:
         return "Keine Aufträge", 404
     
@@ -32,7 +36,7 @@ def add_new_entries():
     db.get_db().commit()
 
     # Reset the cart
-    current_user.data.orders = []
+    session["cart"] = []
 
     return redirect(request.referrer)
 
@@ -44,7 +48,9 @@ def delete_order():
     except:
         return "Incomplete or mistyped form", 400
 
-    del current_user.data.orders[order_index]
+    cart = session.get("cart", default=[])
+    del cart[order_index]
+    session["cart"] = cart
 
     return redirect(request.referrer)
 
@@ -70,7 +76,9 @@ def new_order():
         except:
             return f"Incomplete or mistyped form", 400
         
-        current_user.data.orders.append({
+        cart = session.get("cart", default=[])
+        print(cart)
+        cart.append({
             "barcode": barcode,
             "name": name,
             "sales_units": sales_units,
@@ -83,6 +91,7 @@ def new_order():
             "net_unit_price": net_unit_price,
             "gross_unit_price": gross_unit_price
         })
+        session["cart"] = cart
         return redirect("/entry")
 
     with db.run_query("entry/get_groups.sql") as cursor:
@@ -94,13 +103,10 @@ def new_order():
     with db.run_query("entry/get_tax_groups.sql") as cursor:
         tax_groups = cursor.fetchall()
 
-    selected_location = current_user.data.location
-
     return render_template(
         "entry/new-order.html",
         groups=groups,
         locations=locations,
-        default_location=next(location for location in locations if location["id"] == selected_location["location_id"]) if selected_location is not None else locations[0],
         tax_groups=tax_groups
     )
 
@@ -116,9 +122,10 @@ def api_search_items():
     except:
         return {"error": "Missing query parameter `search-term`"}, 400
 
-    location = current_user.data.location
+    location = session.get("location", None)
+
     with db.run_query("search_items.sql", {
-        "location_id": location["location_id"] if location else None,
+        "location_id": None if location is None else location["location_id"],
         "search_term": search_term
     }) as cursor:
         items = cursor.fetchall()

jon/inventory.py

@@ -1,5 +1,4 @@
-from flask import Blueprint, redirect, render_template, request
-from flask_login import current_user
+from flask import Blueprint, redirect, render_template, request, session
 
 from . import db
 
@@ -9,7 +8,7 @@ bp = Blueprint("inventory", __name__, url_prefix="/inventory")
 
 @bp.get("/")
 def index():
-    location = current_user.data.location
+    location = session.get("location", None)
     items = db.run_query("get_inventory_overview.sql", {
         "location_id": None if location is None else location["location_id"]
     }).fetchall()
@@ -21,18 +20,12 @@ def index():
 
 @bp.get("/report")
 def read_report():
-    location = current_user.data.location
-
-    if location is None:
-        # TODO: Error handling
-        return "please select a location in order to generate a report", 400
-
+    location = session.get("location", None)
     items = db.run_query("get_inventory_report.sql", {
-        "location_id": location["location_id"]
+        "location_id": None if location is None else location["location_id"]
     }).fetchall()
 
     return render_template("inventory/read_report.html", **{
-        "location": location,
         "items": items
     })
 

jon/location.py

@@ -1,5 +1,4 @@
-from flask import Blueprint, render_template, request
-from flask_login import current_user
+from flask import Blueprint, render_template, request, session
 
 from . import db
 
@@ -12,12 +11,13 @@ def index():
     if request.method == "POST":
         location_id = request.form.get("location_id", "")
         if location_id == "":
-            current_user.data.location = None
+            session.pop("location", None)
         else:
             location = db.run_query("get_location_by_id.sql", {
-                "location_id": location_id
-            }).fetchone()
-            current_user.data.location = location
+                "location_id": location_id}
+            ).fetchone()
+            session["location"] = location
+        
 
     locations = db.run_query("get_locations.sql").fetchall()
 

jon/static/jon.css

@@ -40,9 +40,6 @@ nav > ul > li + li:before {
 .--centered {
   text-align: center;
 }
-.--not-important {
-  color: #aaa;
-}
 @keyframes wiggle {
   0%, 100% { margin-top: 0; }
   50% { margin-top: -0.5em; }
@@ -58,10 +55,6 @@ th {
   body {
     font-size: 8px;
   }
-  /* hide the menu when printing */
-  header {
-    display: none;
-  }
 }
 .form-input > label {
   font-size: .8em;
@@ -78,10 +71,3 @@ th {
   display: block;
   width: 8em;
 }
-details {
-  font-size: 0.8em;
-}
-.consumption-graph {
-  display: block;
-  height: 1em;
-}

jon/templates/base.html

@@ -12,14 +12,13 @@
         <ul>
           <li{{ " class=current-page" if request.path == "/" else "" }}><a href="/">Home</a></li>
           <li{{ " class=current-page" if request.path.startswith("/inventory") else "" }}><a href="/inventory">Inventar</a></li>
-          <li><a href="/inventory/report">Einkäuferbericht</a></li>
           <li{{ " class=current-page" if request.path.startswith("/entry") else "" }}><a href="/entry">Eintragen</a></li>
           <li{{ " class=current-page" if request.path.startswith("/location") else "" }}>
             <a href="/location">
-            {% if not current_user.data.location %}
+            {% if "location" not in session %}
               Raum wählen
             {% else %}
-              Raum: {{ current_user.data.location.location_name }}
+              Raum: {{ session.location.location_name }}
             {% endif %}
             </a>
           </li>
@@ -31,16 +30,6 @@
       <details>
         <summary><code>config</code></summary>
         <pre>{% for key, value in config.items() %}{{ key }} = {{ value }}
-{% endfor %}</pre>
-      </details>
-      <details>
-        <summary><code>session</code></summary>
-        <pre>{% for key, value in session.items() %}{{ key }} = {{ value }}
-{% endfor %}</pre>
-      </details>
-      <details>
-        <summary><code>current_user.data</code></summary>
-        <pre>{% for key, value in current_user.data.__dict__.items() %}{{ key }} = {{ value }}
 {% endfor %}</pre>
       </details>
       {% endif %}

jon/templates/entry/index.html

@@ -15,7 +15,7 @@
     <th>VK-Preis (Brutto)</th>
     <th>Aktionen</th>
   </tr>
-  {% for cart_item in current_user.data.orders %}
+  {% for cart_item in cart %}
   <tr>
     <td><code>{{ cart_item.barcode }}</code></td>
     <td>{{ cart_item.name }}</td>

jon/templates/entry/new-order.html

@@ -8,11 +8,8 @@ Elm.Entry.init({
   node: document.querySelector('.entry-app'),
   flags: {
     locations: {{ to_json(locations) | safe }},
-    defaultLocation: {{ to_json(default_location) | safe }},
     groups: {{ to_json(groups) | safe }},
-    defaultGroup: {{ to_json(groups[0]) | safe }},
-    taxGroups: {{ to_json(tax_groups) | safe }},
-    defaultTaxGroup: {{ to_json(tax_groups[0]) | safe }},
+    taxGroups: {{ to_json(tax_groups) | safe }}
   }
 });
 </script>

jon/templates/inventory/index.html

@@ -1,34 +1,9 @@
-{% macro consumption_graph_svg(values) -%}
-{% set stroke_width = 8 %}
-{% set width = 300 %}
-{% set height = 100 %}
-{% set padding = 4 %}
-{% set dx = (width - 2 * padding) / ((values | length) + 1) %}
-{% set dy = (height - 2 * padding) / ((values + [1]) | max) %}
-<svg viewBox="0 0 {{ width }} {{ height }}" role="img" class="consumption-graph">
-  <polyline
-    points="
-    {% for value in values %}
-      {{ padding + loop.index * dx }}, {{ height - padding - value * dy }}
-    {% endfor %}
-    "
-    stroke="green"
-    stroke-width="{{ stroke_width }}"
-    stroke-linecap="round"
-    stroke-linejoin="round"
-    fill="none"
-  >
-</svg>
-{% endmacro -%}
-
 {% extends "base.html" %}
 
 {% block content %}
 <table>
   <tr>
-    <th>Stat</th>
     <th>ID</th>
-    <th>Graph</th>
     <th>Barcode</th>
     <th>Name</th>
     <th>Preis (Netto)</th>
@@ -42,12 +17,7 @@
   </tr>
 {% for item in items %}
   <tr>
-    <td>
-      {% if item.units_left == 0 %}<span title="Leerer aktiver Inventareintrag">🅾️</span>{% endif %}
-      {% if item.other_lines_count != 0 %}<span title="{% if item.other_lines_count == 1 %}Neuerer Eintrag mit demselben Barcode ist aktiv{% else %}{{ item.other_lines_count }} Einträge mit demselben Barcode sind aktiv{% endif %}">🔄</span>{% endif %}
-    </td>
     <td><a href="/inventory/item/{{ item.item_id }}">{{ item.item_id }}</a></td>
-    <td>{{ consumption_graph_svg(item.last_n_days_sales) }}</td>
     <td><code>{{ item.item_barcode }}</code></td>
     <td>{{ item.name }}</td>
     <td class="--align-right">{{ format_currency(item.unit_price) }}</td>

jon/templates/inventory/read_report.html

@@ -1,29 +1,29 @@
 {% extends "base.html" %}
 
 {% block content %}
-<h2>Einkäuferbericht für {{ location.location_name }}</h2>
-
 <table>
   <tr>
+    <th>ID</th>
     <th>Barcode</th>
     <th>Name</th>
     <th>Inventar</th>
-    <th title="In den letzten 4 Monaten verkauft">Verkauft</th>
+    <th>Gesamt</th>
+    <th>Raum</th>
     <th>Verbrauch [1/d]</th>
-    <th>Verbrauch [1/60d]</th>
-    <!--
     <th title="Estimated Time Until Empty">ETUE [d]</th>
     <th>Für 2m</th>
-    -->
   </tr>
 {% for item in items %}
-<tr{% if item.units_left >= item.units_sold %} class="--not-important"{% endif %}>
+  <tr>
+    <td><a href="/inventory/item/{{ item.item_id }}">{{ item.item_id }}</a></td>
     <td><code>{{ item.item_barcode }}</code></td>
     <td>{{ item.name }}</td>
     <td class="--align-right">{{ item.units_left }}</td>
-    <td class="--align-right">{{ item.units_sold }}</td>
-    <td class="--align-right">{{ item.sold_per_day }}</td>
-    <td class="--align-right">{{ item.sold_per_day * 60 }}</td>
+    <td class="--align-right">{{ item.sales_units + item.correction_delta }}</td>
+    <td>{{ item.location_name }}</td>
+    <td class="--align-right">{{ item.per_day|round(2) }}</td>
+    <td class="--align-right">{% if item.days_left != None %}{{ item.days_left|round(1) }}{% endif %}</td>
+    <td class="--align-right">{% if item.for_two_months %}{{ item.for_two_months|round(1) }}{% endif %}</td>
   </tr>
 {% endfor %}
 </table>

jon/templates/location/index.html

@@ -3,9 +3,9 @@
 {% block content %}
 <form method="POST">
   <select name="location_id">
-    <option value="" {{ "selected" if not current_user.data.location else ""}}>-</option>
+    <option value="" {{ "selected" if "location" not in session else ""}}>-</option>
   {% for location in locations %}
-    <option value="{{ location.location_id }}" {{ "selected" if current_user.data.location.location_id == location.location_id else "" }}>{{ location.location_name }}</option>
+    <option value="{{ location.location_id }}" {{ "selected" if "location" in session and session.location.location_id == location.location_id else "" }}>{{ location.location_name }}</option>
   {% endfor %}
   </select>
 

requirements.txt

@@ -1,7 +1,6 @@
 blinker==1.6.2
 click==8.1.3
 Flask==2.3.2
-Flask-Login==0.6.2
 itsdangerous==2.1.2
 Jinja2==3.1.2
 MarkupSafe==2.1.2