rewrite topics routes to include id

app/models.py

@@ -164,6 +164,7 @@ class Topics(Model):
                 GROUP BY thread_id
             )
             SELECT
+                threads.id,
                 threads.title,
                 threads.slug,
                 threads.created_at,

app/routes/mod.py

@@ -21,7 +21,7 @@ def new_topic():
 @bp.post('/topics/new/')
 def new_topic_post():
     topic = Topics.new(request.form.get('name'), request.form.get('description'))
-    return redirect(url_for('topics.topic', slug=topic.slug))
+    return redirect(url_for('topics.topic_by_id', topic_id=topic.id))
 
 @bp.get('/topics/sort/')
 def sort_topics():
@@ -43,7 +43,7 @@ def edit_topic_post(topic_id):
         'name': request.form.get('name').strip(),
         'description': request.form.get('description').strip(),
     })
-    return redirect(url_for('topics.topic', slug=topic.slug))
+    return redirect(url_for('topics.topic_by_id', topic_id=topic.id))
 
 @bp.post('/topics/<int:topic_id>/lock/')
 def lock_topic(topic_id):
@@ -51,7 +51,7 @@ def lock_topic(topic_id):
     if not topic:
         abort(404)
     topic.update({'is_locked': request.form.get('lock', default=0)})
-    return redirect(url_for('topics.topic', slug=topic.slug))
+    return redirect(url_for('topics.topic_by_id', topic_id=topic.id))
 
 @bp.post('/threads/<int:thread_id>/move/')
 def move_thread(thread_id):
@@ -62,7 +62,7 @@ def move_thread(thread_id):
     if not target_topic:
         abort(404)
     thread.update({'topic_id': target_topic.id})
-    return redirect(url_for('threads.thread', slug=thread.slug))
+    return redirect(url_for('threads.thread_by_id', thread_id=thread.id))
 
 @bp.post('/threads/<int:thread_id>/lock/')
 def lock_thread(thread_id):
@@ -70,7 +70,7 @@ def lock_thread(thread_id):
     if not thread:
         abort(404)
     thread.update({'is_locked': request.form.get('lock')})
-    return redirect(url_for('threads.thread', slug=thread.slug))
+    return redirect(url_for('threads.thread_by_id', thread_id=thread.id))
 
 @bp.post('/threads/<int:thread_id>/sticky/')
 def sticky_thread(thread_id):
@@ -78,7 +78,7 @@ def sticky_thread(thread_id):
     if not thread:
         abort(404)
     thread.update({'is_stickied': request.form.get('sticky')})
-    return redirect(url_for('threads.thread', slug=thread.slug))
+    return redirect(url_for('threads.thread_by_id', thread_id=thread.id))
 
 @bp.post('/users/<int:user_id>/make-guest/')
 @csrf_verified

app/routes/posts.py

@@ -1,7 +1,44 @@
-from flask import Blueprint
+from flask import Blueprint, abort
+from functools import wraps
+from ..auth import login_required, get_active_user
+from ..models import Posts
 
 bp = Blueprint('posts', __name__, url_prefix='/posts/')
 
+def ownership_required(view_func):
+    @wraps(view_func)
+    def wrapper(*args, **kwargs):
+        post = Posts.find({'id': kwargs.get('post_id', None)})
+        if not post:
+            abort(404)
+
+        if post.user_id != get_active_user().id:
+            abort(403)
+
+        return view_func(*args, **kwargs)
+    return wrapper
+
+def ownership_or_mod_required(view_func):
+    @wraps(view_func)
+    def wrapper(*args, **kwargs):
+        post = Posts.find({'id': kwargs.get('post_id', None)})
+        if not post:
+            abort(404)
+
+        if post.user_id != get_active_user().id and not get_active_user().is_mod():
+            abort(403)
+
+        return view_func(*args, **kwargs)
+    return wrapper
+
 @bp.get('/<int:post_id>/edit/')
+@login_required
+@ownership_required
 def edit(post_id):
     return 'stub'
+
+@bp.get('/<int:post_id>/delete/')
+@login_required
+@ownership_or_mod_required
+def delete(post_id):
+    return 'stub'

app/routes/threads.py

@@ -5,11 +5,21 @@ import math
 
 bp = Blueprint('threads', __name__, url_prefix='/threads/')
 
-@bp.get('/<slug>/')
-def thread(slug):
-    thread = Threads.find({'slug': slug})
+@bp.get('/<int:thread_id>/')
+def thread_by_id(thread_id):
+    thread = Threads.find({'id': thread_id})
     if not thread:
         abort(404)
+    return redirect(url_for('.thread', thread_id=thread_id, slug=thread.slug, **request.args))
+
+@bp.get('/<int:thread_id>/<slug>/')
+def thread(thread_id, slug):
+    thread = Threads.find({'id': thread_id})
+    if not thread:
+        abort(404)
+    if thread.slug != slug:
+        return redirect(url_for('.thread', thread_id=thread_id, slug=thread.slug, **request.kwargs))
+
     topic = Topics.find({'id': thread.topic_id})
     started_by = Users.find({'id': thread.user_id})
     PER_PAGE = 10
@@ -34,21 +44,21 @@ def thread(slug):
             abort(404)
     return render_template('threads/thread.html', thread=thread, posts=thread.get_posts(PER_PAGE, page), page=page, page_count=page_count, topic=topic, started_by=started_by, topics=Topics.get_list(), Reactions=Reactions)
 
-@bp.post('/<slug>/reply/')
+@bp.post('/<int:thread_id>/reply/')
 @login_required
-def reply(slug):
+def reply(thread_id):
     user = get_active_user()
-    thread = Threads.find({'slug': slug})
+    thread = Threads.find({'id': thread_id})
     if not thread:
         abort(404)
     if thread.locked() and not user.is_mod():
         # TODO: flash
-        return redirect(url_for('.thread', slug=slug))
+        return redirect(url_for('.thread_by_id', thread_id=thread_id))
     post = Posts.new(user.id, thread.id, request.form.get('babycode_content'))
-    return redirect(url_for('.thread', slug=slug, after=post.id, _anchor=f'post-{post.id}'))
+    return redirect(url_for('.thread_by_id', thread_id=thread_id, after=post.id, _anchor=f'post-{post.id}'))
 
-@bp.get('/<slug>/feed.atom/')
-def feed(slug):
+@bp.get('/<int:thread_id>/feed.atom/')
+def feed(thread_id):
     return 'stub'
 
 @bp.get('/new/')

app/routes/topics.py

@@ -10,13 +10,23 @@ def all_topics():
     topic_list = Topics.get_list()
     return render_template('topics/topics.html', topics=topic_list)
 
-@bp.get('/<slug>/')
-def topic(slug):
-    topic = Topics.find({'slug': slug})
+@bp.get('/<int:topic_id>/')
+def topic_by_id(topic_id):
+    topic = Topics.find({'id': topic_id})
     if not topic:
         abort(404)
+    return redirect(url_for('.topic', topic_id=topic_id, slug=topic.slug, **request.args))
+
+@bp.get('/<int:topic_id>/<slug>/')
+def topic(topic_id, slug):
+    topic = Topics.find({'id': topic_id})
+    if not topic:
+        abort(404)
+    if topic.slug != slug:
+        return redirect(url_for('.topic', topic_id=topic_id, slug=topic.slug, **request.args))
+
     sort_by = request.args.get('sort_by', default=session.get('sort_by', default='activity'))
-    PER_PAGE = 3
+    PER_PAGE = 10
     threads_count = Threads.count({'topic_id': topic.id})
     page_count = max(1, math.ceil(threads_count / PER_PAGE))
     try:
@@ -25,6 +35,6 @@ def topic(slug):
         abort(404)
     return render_template('topics/topic.html', topic=topic, threads=topic.get_threads(PER_PAGE, page, sort_by), sort_by=sort_by, page=page, page_count=page_count)
 
-@bp.get('/<slug>/feed.atom')
-def feed(slug):
+@bp.get('/<int:topic_id>/feed.atom/')
+def feed(topic_id):
     return 'stub'

app/templates/common/macros.html

@@ -147,18 +147,18 @@
       <a class="linkbutton" href="{{url_for('posts.edit', post_id=post.id)}}">Edit</a>
       {%- endif -%}
       {%- if can_reply -%}
-      <button>Quote</button>
+      <button disabled title="This feature requires JavaScript to be enabled.">Quote</button>
       {%- endif -%}
       {%- if can_delete -%}
-      <button class="critical">Delete</button>
+      <a class="linkbutton critical" href="{{url_for('posts.delete', post_id=post.id)}}">Delete</a>
       {%- endif -%}
-      <button>Bookmark&hellip;</button>
+      <button disabled title="This feature requires JavaScript to be enabled.">Bookmark&hellip;</button>
     </span>
     {%- endif -%}
   </div>
   <div class="plank even no-shadow post-content-inner minimal">{{post.content | safe}}
     {%- if render_sig and post.signature_rendered -%}
-    <div class="post-signature">{{post.signature_rendered | safe}}</div>
+    <aside class="post-signature">{{post.signature_rendered | safe}}</aside>
     {%- endif -%}
   </div>
   {%- if show_reactions -%}
@@ -172,10 +172,10 @@
         {% set reactors_str = reactors_str + '\n...and many others' %}
       {% endif %}
       {% set has_reacted = get_active_user() is not none and get_active_user().username in reactors %}
-      <button {{'disabled' if not is_logged_in() else ''}} title="{{reactors_str}}" class="minimal {{'alt' if has_reacted else ''}}"><img src="/static/emoji/{{reaction.reaction_text}}.png">{{reaction.c}}</button>
+      <button disabled title="{{reactors_str}}" class="minimal {{'alt' if has_reacted else ''}}"><img src="/static/emoji/{{reaction.reaction_text}}.png">{{reaction.c}}</button>
       {%- endfor -%}
     </span>
-    {%- if is_logged_in() -%}<button>Add reaction</button>{%- endif -%}
+    {%- if is_logged_in() -%}<button disabled title="This feature requires JavaScript to be enabled.">Add reaction</button>{%- endif -%}
   </div>
   {%- endif -%}
 </div>

app/templates/threads/thread.html

@@ -5,7 +5,7 @@
 {%- block content -%}
 {%- set td -%}
 <ul class="horizontal">
-<li>Started by <a href="{{url_for('users.user_page', username=started_by.username)}}">{{started_by.get_readable_name()}}</a> in topic <a href="{{url_for('topics.topic', slug=topic.slug)}}">{{topic.name}}</a></li>
+<li>Started by <a href="{{url_for('users.user_page', username=started_by.username)}}">{{started_by.get_readable_name()}}</a> in topic <a href="{{url_for('topics.topic_by_id', topic_id=topic.id)}}">{{topic.name}}</a></li>
 {%- if thread.locked() or thread.stickied() -%}
   {%- if thread.locked() -%}
   <li class="visible">Locked</li>
@@ -21,9 +21,9 @@
   <legend>Actions</legend>
   {%- if is_logged_in() -%}
   <button>Subscribe</button>
-  <button>Bookmark&hellip;</button>
+  <button disabled title="This feature requires JavaScript to be enabled.">Bookmark&hellip;</button>
   {%- endif -%}
-  <a href="{{url_for('threads.feed', slug=thread.slug)}}" class="linkbutton rss">Subscribe via RSS</a>
+  <a href="{{url_for('threads.feed', thread_id=thread.id)}}" class="linkbutton rss">Subscribe via RSS</a>
 </fieldset>
 {%- if is_mod() -%}
 <fieldset class="plank even no-shadow minimal thread-actions">
@@ -35,9 +35,10 @@
     <input type="submit" class="warn" value="{{'Unsticky' if thread.stickied() else 'Sticky'}}" formaction="{{url_for('mod.sticky_thread', thread_id=thread.id)}}">
   </form>
   <form class="horizontal wrap" method="POST" action="{{url_for('mod.move_thread', thread_id=thread.id)}}">
-    <select name="new_topic_id" id="new-topic-id">
+    <select name="new_topic_id" id="new-topic-id" autocomplete="off" required>
+      <option selected disabled value="">Move to topic:</option>
       {%- for t in topics -%}
-      <option value="{{t.id}}" {{'selected disabled' if t.id == topic.id else ''}} autocomplete="off">{{t.name}}</option>
+      <option value="{{t.id}}" {{'disabled' if t.id==topic.id else ''}}>{{t.name}}</option>
       {%- endfor -%}
     </select>
     <input type="submit" value="Move" class="warn">
@@ -63,7 +64,7 @@
   </fieldset>
 </div>
 {%- if is_logged_in() -%}
-<form action="{{url_for('threads.reply', slug=thread.slug)}}" method="POST" class="plank post-edit-form">
+<form action="{{url_for('threads.reply', thread_id=thread.id)}}" method="POST" class="plank post-edit-form">
   <h2 class="info">Reply to "{{thread.title}}"</h2>
   {{- babycode_editor_component() -}}
   <span>

app/templates/topics/topic.html

@@ -16,7 +16,7 @@
   {%- if is_logged_in() and get_active_user().can_post_to_topic(topic) -%}
   <a href="{{url_for('threads.new', topic_id=topic.id)}}" class="linkbutton">New thread</a>
   {%- endif -%}
-  <a href="{{url_for('topics.feed', slug=topic.slug)}}" class="linkbutton rss">Subscribe via RSS</a>
+  <a href="{{url_for('topics.feed', topic_id=topic.id)}}" class="linkbutton rss">Subscribe via RSS</a>
   <form method="GET">
     <select name="sort_by">
       <option value="activity"{% if sort_by == 'activity' %}selected{% endif %}>Sorted by activity</option>
@@ -48,10 +48,10 @@
 {%- for thread in threads -%}
 <div class="topic-info plank">
   <div class="title-container">
-    <span class="info thread-title-counter"><a href="{{url_for('threads.thread', slug=thread.slug)}}">{{thread.title}}</a></span>
+    <span class="info thread-title-counter"><a href="{{url_for('threads.thread_by_id', thread_id=thread.id)}}">{{thread.title}}</a></span>
     <ul class="horizontal"></ul>
     {%- if thread.posts_count / 10 > 1 -%}
-    {{pager(0, (((thread.posts_count / 10) | round(0, 'ceil') )| int), 'flex-last', url=url_for('threads.thread', slug=thread.slug))}}
+    {{pager(0, (((thread.posts_count / 10) | round(0, 'ceil') )| int), 'flex-last', url=url_for('threads.thread_by_id', thread_id=thread.id))}}
     {%- endif -%}
   </div>
   <span>Started by <a href="{{url_for('users.user_page', username=thread.started_by)}}">{{thread.started_by_display_name if thread.started_by_display_name else thread.started_by}}</a> on {{timestamp(thread.created_at)}}</span>

app/templates/topics/topics.html

@@ -13,7 +13,7 @@
 {%- for topic in topics -%}
 <div class="topic-info plank">
   <div class="title-container">
-    <a class="info" href="{{url_for('topics.topic', slug=topic.slug)}}">{{topic.name}}</a>
+    <a class="info" href="{{url_for('topics.topic_by_id', topic_id=topic.id)}}">{{topic.name}}</a>
   </div>
   <div>{{topic.description}}</div>
   <ul class="horizontal">

app/util.py

@@ -11,7 +11,7 @@ def get_post_url(post_id, _anchor=False, external=False):
 
     anchor = None if not _anchor else f'post-{post_id}'
 
-    return url_for('threads.thread', slug=thread.slug, after=post_id, _external=external, _anchor=anchor)
+    return url_for('threads.thread_by_id', thread_id=thread.id, after=post_id, _external=external, _anchor=anchor)
 
 def dict_to_query_string(d) -> str:
     return '?' + '&'.join([f'{key}={str(value)}' for key, value in d.items()])

data/static/css/style.css

@@ -108,6 +108,8 @@ button, .linkbutton, input[type="submit"] {
   text-decoration: none;
   user-select: none;
 
+  cursor: pointer;
+
   line-height: normal;
   display: inline flex;
   align-items: center;
@@ -322,6 +324,20 @@ fieldset {
   margin: 0;
 }
 
+legend {
+  padding: var(--small-padding);
+  border-radius: var(--border-radius);
+  border: 2px groove var(--border-color);
+  margin-top: var(--small-padding);
+
+  .plank:not(.secondary-bg) > & {
+    background-color: var(--bg-color-secondary);
+  }
+  .plank.secondary-bg > & {
+    background-color: var(--bg-color-primary);
+  }
+}
+
 ul.horizontal, ol.horizontal {
   display: inline flex;
   align-items: center;