freak/__init__.py

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

freak/rest/__init__.py

@@ -446,11 +446,7 @@ 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="3">Put on hold</button>
+    <button type="submit" name="do" value="2">Put on hold</button>
   </form>
 {% endblock %}

freak/website/admin.py

@@ -9,7 +9,6 @@ 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
 
@@ -98,7 +97,8 @@ def get_content(target) -> str | None:
 REPORT_ACTIONS = {}
 
 @additem(REPORT_ACTIONS, '1')
-async def accept_report(target, source: PostReport, session: AsyncSession):
+async def accept_report(target, source: PostReport):
+    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,17 +106,18 @@ async def accept_report(target, source: PostReport, session: AsyncSession):
         await remove_content(target, source.reason_code)
 
         source.update_status = REPORT_UPDATE_COMPLETE
-    session.add(source)
+        # XXX disabled because of a session conflict
-    await session.commit()
+        #session.add(source)
 
 
 @additem(REPORT_ACTIONS, '2')
-async def strike_report(target, source: PostReport, session: AsyncSession):
+async def strike_report(target, source: PostReport):
+    async with db as session:
         await remove_content(target, source.reason_code)
 
         author = get_author(target)
         if author:
-        await session.execute(insert(UserStrike).values(
+            session.execute(insert(UserStrike).values(
                 user_id = author.id,
                 target_type = TARGET_TYPES[type(target)],
                 target_id = target.id,
@@ -130,22 +131,21 @@ async def strike_report(target, source: PostReport, session: AsyncSession):
                 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, session: AsyncSession):
+async def reject_report(target, source: PostReport):
+    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, session: AsyncSession):
+async def withhold_report(target, source: PostReport):
+    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, session)
+            await action(target, report)
             return redirect(url_for('admin.reports'))
     return await render_template('admin/admin_report_detail.html', report=report,
         report_reasons=REPORT_REASON_STRINGS)