fix admin board not accepting reports due to session issues

freak/__init__.py

@@ -26,7 +26,7 @@ from suou import twocolon_list, WantsContentType
 
 from .colors import color_themes, theme_classes
 
-__version__ = '0.5.0-dev49'
+__version__ = '0.5.0-dev50'
 
 APP_BASE_DIR = os.path.dirname(os.path.dirname(__file__))
 

freak/rest/__init__.py

@@ -446,7 +446,11 @@ async def patch_settings_appearance(data: SettingsIn):
 @bp.get('/about/about')
 async def about_about():
     return dict(
-        content=await render_template("about.md")
+        content=await render_template("about.md",
+            quart_version=quart_version,
+            sa_version=sa_version,
+            python_version=sys.version.split()[0]
+        )
     )
 
 @bp.get('/about/terms')

freak/templates/admin/admin_report_detail.html

@@ -29,6 +29,6 @@
     <button type="submit" name="do" value="1" class="primary">Remove</button>
     <button type="submit" name="do" value="2">Strike</button>
     {% endif %}
-    <button type="submit" name="do" value="2">Put on hold</button>
+    <button type="submit" name="do" value="3">Put on hold</button>
   </form>
 {% endblock %}

freak/website/admin.py

@@ -9,6 +9,7 @@ from quart import Blueprint, abort, redirect, render_template, request, send_fro
 from quart_auth import current_user
 from markupsafe import Markup
 from sqlalchemy import insert, select, update
+from sqlalchemy.ext.asyncio import AsyncSession
 from suou import additem, not_implemented
 import logging
 
@@ -97,8 +98,7 @@ def get_content(target) -> str | None:
 REPORT_ACTIONS = {}
 
 @additem(REPORT_ACTIONS, '1')
-async def accept_report(target, source: PostReport):
+async def accept_report(target, source: PostReport, session: AsyncSession):
-    async with db as session:
     if source.is_critical():
         warnings.warn('attempted remove on a critical report case, striking instead', UserWarning)
         return await strike_report(target, source)
@@ -106,18 +106,17 @@ async def accept_report(target, source: PostReport):
     await remove_content(target, source.reason_code)
 
     source.update_status = REPORT_UPDATE_COMPLETE
-        # XXX disabled because of a session conflict
+    session.add(source)
-        #session.add(source)
+    await session.commit()
 
 
 @additem(REPORT_ACTIONS, '2')
-async def strike_report(target, source: PostReport):
+async def strike_report(target, source: PostReport, session: AsyncSession):
-    async with db as session:
     await remove_content(target, source.reason_code)
 
     author = get_author(target)
     if author:
-            session.execute(insert(UserStrike).values(
+        await session.execute(insert(UserStrike).values(
             user_id = author.id,
             target_type = TARGET_TYPES[type(target)],
             target_id = target.id,
@@ -131,21 +130,22 @@ async def strike_report(target, source: PostReport):
             author.banned_reason = source.reason_code
 
     source.update_status = REPORT_UPDATE_COMPLETE
-        #session.add(source)
+    session.add(source)
+    await session.commit()
 
 
 @additem(REPORT_ACTIONS, '0')
-async def reject_report(target, source: PostReport):
+async def reject_report(target, source: PostReport, session: AsyncSession):
-    async with db as session:
     source.update_status = REPORT_UPDATE_REJECTED
-        #session.add(source)
+    session.add(source)
+    await session.commit()
 
 
 @additem(REPORT_ACTIONS, '3')
-async def withhold_report(target, source: PostReport):
+async def withhold_report(target, source: PostReport, session: AsyncSession):
-    async with db as session:
     source.update_status = REPORT_UPDATE_ON_HOLD
-        #session.add(source)
+    session.add(source)
+    await session.commit()
 
 
 @additem(REPORT_ACTIONS, '4')
@@ -184,7 +184,7 @@ async def report_detail(id: int):
         if request.method == 'POST':
             form = await get_request_form()
             action = REPORT_ACTIONS[form['do']]
-            await action(target, report)
+            await action(target, report, session)
             return redirect(url_for('admin.reports'))
         return await render_template('admin/admin_report_detail.html', report=report,
             report_reasons=REPORT_REASON_STRINGS)