pin werkzeug to version 2.2.2 to prevent incompatibilities

Reviewed-on: http://gitea.lusiardi.de/jlusiardi/rollerverbrauch/pulls/7

app/__init__.py

@@ -6,7 +6,6 @@ from flask_sqlalchemy import SQLAlchemy
 import os
 from config import config
 from flask_security.forms import LoginForm
-from flask_limiter import Limiter
 from flask_limiter.util import get_remote_address
 
 from .forms import *
@@ -15,13 +14,6 @@ from .forms import *
 app = Flask(__name__)
 app.config.from_object(config[os.getenv('FLASK_CONFIG') or 'default'])
 
-# applies to all routes, so choose limits wisely!
-limiter = Limiter(
-    app,
-    key_func=get_remote_address,
-#    default_limits=["500 per second"]
-)
-
 
 @app.errorhandler(429)
 def ratelimit_handler(e):

app/forms/service.py

@@ -6,7 +6,7 @@ from .checks import *
 
 
 class CreateServiceForm(FlaskForm):
-    date = DateField('Date of Pitstop')
+    date = DateField('Date of Service')
     odometer = IntegerField('Odometer (km)', validators=[odometer_date_check])
     costs = DecimalField('Costs (€, overall)', places=2, validators=[costs_check])
     description = TextAreaField('Description', validators=[Length(1, 4096)])
@@ -72,4 +72,4 @@ class EditServiceForm(FlaskForm):
             'litres': 'Litres must be higher than 0.01 L.',
             'costs': 'Costs must be higher than 0.01 €.'
         }
-        return messages
+        return messages

app/routes/filling_stations.py

@@ -4,7 +4,7 @@ from flask_security.core import current_user
 import requests
 
 from ..entities import FillingStation
-from .. import app, db, limiter
+from .. import app, db
 
 
 @app.route('/filling_stations/favourites/toggle/<fsid>')
@@ -24,7 +24,6 @@ def add_favourite_filling_stations(fsid):
 
 @app.route('/filling_stations', methods=['GET'])
 @login_required
-@limiter.limit('1 per second')
 def query_filling_stations():
     api_key = app.config['TANKERKOENIG_API_KEY']
 

app/routes/service.py

@@ -14,6 +14,7 @@ from ..tools import (
     db_log_delete,
     get_event_line_for_vehicle,
     get_latest_pitstop_for_vehicle,
+    get_users_active_vehicle,
 )
 from .. import app, db
 

app/templates/statistics.html

@@ -164,6 +164,7 @@
                                 <ul id="consumable_{{vehicle.id}}_{{consumable.id}}_tabs" class="nav nav-tabs" data-tabs="tabs">
                                     {{ nav_tab(vehicle.id|string + '_' + consumable.id|string + '_consumption', 'Consumption', true) }}
                                     {{ nav_tab(vehicle.id|string + '_' + consumable.id|string + '_amount', 'Amount', false) }}
+                                    {{ nav_tab(vehicle.id|string + '_' + consumable.id|string + '_price', 'Price', false) }}
                                 </ul>
                                 <div id="consumable_{{vehicle.id}}_{{consumable.id}}_content" class="tab-content ">
                                     {{ tab_pane(
@@ -188,6 +189,17 @@
                                         false
                                         )
                                     }}
+                                    {{ tab_pane(
+                                        vehicle.id|string + '_' + consumable.id|string + '_price',
+                                        chart(
+                                            consumable.price,
+                                            'ref_' + vehicle.id|string + '_' + consumable.id|string + '_price',
+                                            '€ / '+consumable.unit,
+                                            url_for('create_pit_stop_form', vid=vehicle.id, cid=consumable.id)
+                                            ),
+                                        false
+                                        )
+                                    }}
                                 </div>
                                 {{ tab_script('vehicle_' + vehicle.id|string + '_' + consumable.id|string + '_tabs') }}
                             </div>

app/tools.py

@@ -27,6 +27,7 @@ class ConsumableStats:
         self.average_amount_used = 0
         self.average_amount = []
         self.amounts = []
+        self.price = []
 
         pitstops = [
             stop for stop in vehicle.pitstops if stop.consumable_id == consumable.id
@@ -37,6 +38,9 @@ class ConsumableStats:
             for pitstop in pitstops:
                 self.overall_amount += pitstop.amount
                 self.amounts.append(StatsEvent(pitstop.date, pitstop.amount))
+                # some pitstops seem to have lost their costs...
+                if pitstop.costs:                
+                    self.price.append(StatsEvent(pitstop.date, pitstop.costs/pitstop.amount))
             self.average_amount_fuelled = self.overall_amount / pitstop_count
         if pitstop_count > 1:
             overall_distance = (
@@ -204,7 +208,9 @@ def compute_lower_limits_for_new_pitstop(
 
 
 def pitstop_service_key(x):
-    return x.date, x.odometer
+    # if the entry got no odometer (regular costs!) then we assume it's okay
+    # to have regular cost at a virtual odometer of 0 on that day.
+    return x.date, x.odometer or 0
 
 
 def get_event_line_for_vehicle(vehicle):

config.py

@@ -41,8 +41,8 @@ class TestingConfig(Config):
 
 
 class ProductionConfig(Config):
-    SQLALCHEMY_DATABASE_URI = 'mysql+pymysql://root:{h}@database/pitstops'.format(
-        h=os.environ.get('DATABASE_ENV_MYSQL_ROOT_PASSWORD'))
+    SQLALCHEMY_DATABASE_URI = 'mysql+pymysql://pitstops:{h}@localhost/pitstops'.format(
+        h=os.environ.get('MYSQL_PASSWORD'))
 
 
 config = {

requirements.txt

@@ -1,9 +1,13 @@
-Flask
-Flask-SQLAlchemy
-Flask-Security
-Flask-WTF
-PyMySQL
+Flask==2.1.2
+Flask-SQLAlchemy==2.5.1
+Flask-Security==3.0.0
+Flask-WTF==1.0.1
+PyMySQL==1.0.2
 markdown
-Flask-Limiter
-requests
-email_validator
+Flask-Limiter==2.4.5.1
+requests==2.27.1
+email-validator==1.2.1
+gunicorn==20.1.0
+pytz==2022.1
+SQLAlchemy==1.4.36
+Werkzeug==2.2.2