client/posts: respect edit privileges in sidebar

client/html/post_edit_sidebar.tpl

@@ -1,6 +1,7 @@
 <div class='edit-sidebar'>
     <form autocomplete='off'>
         <div class='input'>
+            <% if (ctx.canEditPostSafety) { %>
                 <section class='safety'>
                     <label>Safety</label>
                     <%= ctx.makeRadio({
@@ -22,14 +23,18 @@
                         class: 'safety-unsafe',
                         text: 'Unsafe'}) %>
                 </section>
+            <% } %>
 
+            <% if (ctx.canEditPostTags) { %>
                 <section class='tags'>
                     <%= ctx.makeTextInput({
                         text: 'Tags',
                         value: ctx.post.tags.join(' '),
-                    readonly: !ctx.canEditPostTags}) %>
+                    }) %>
                 </section>
+            <% } %>
 
+            <% if (ctx.canEditPostRelations) { %>
                 <section class='relations'>
                     <%= ctx.makeTextInput({
                         text: 'Relations',
@@ -37,29 +42,23 @@
                         placeholder: 'space-separated post IDs',
                         pattern: '^[0-9 ]*$',
                         value: ctx.post.relations.map(rel => rel.id).join(' '),
-                    readonly: !ctx.canEditPostRelations}) %>
+                    }) %>
                 </section>
+            <% } %>
 
-            <% if ((ctx.editingNewPost && ctx.canCreateAnonymousPosts) || ctx.post.type === 'video') { %>
+            <% if (ctx.canEditPostFlags && ctx.post.type === 'video') { %>
                 <section class='flags'>
                     <label>Miscellaneous</label>
 
-                    <% if (ctx.editingNewPost && ctx.canCreateAnonymousPosts) { %>
-                        <%= ctx.makeCheckbox({
-                            text: 'Don\'t show me as uploader',
-                            name: 'anonymous'}) %>
-                    <% } %>
-
-                    <% if (ctx.post.type === 'video') { %>
                     <!-- TODO: bind state -->
                     <%= ctx.makeCheckbox({
                         text: 'Loop video',
                         name: 'loop',
-                            readonly: !ctx.canEditPostFlags}) %>
+                    }) %>
-                    <% } %>
                 </section>
             <% } %>
         </div>
+
         <div class='messages'></div>
 
         <div class='buttons'>

client/js/controllers/post_controller.js

@@ -100,9 +100,15 @@ class PostController {
     _evtPostEdit(e) {
         // TODO: disable form
         const post = e.detail.post;
+        if (e.detail.tags !== undefined) {
             post.tags = e.detail.tags;
+        }
+        if (e.detail.safety !== undefined) {
             post.safety = e.detail.safety;
+        }
+        if (e.detail.relations !== undefined) {
             post.relations = e.detail.relations;
+        }
         post.save()
             .then(() => {
                 misc.disableExitConfirmation();

client/js/controls/post_edit_sidebar_control.js

@@ -34,22 +34,30 @@ class PostEditSidebarControl extends events.EventTarget {
             this._formNode.addEventListener('submit', e => this._evtSubmit(e));
         }
 
+        if (this._tagInputNode) {
             this._tagControl = new TagInputControl(this._tagInputNode);
         }
+    }
 
     _evtSubmit(e) {
         e.preventDefault();
         this.dispatchEvent(new CustomEvent('submit', {
             detail: {
                 post: this._post,
-                safety:
+
+                safety: this._safetyButtonNodes.legnth ?
                     Array.from(this._safetyButtonNodes)
                         .filter(node => node.checked)[0]
-                        .value.toLowerCase(),
+                        .value.toLowerCase() :
-                tags:
+                    undefined,
-                    misc.splitByWhitespace(this._tagInputNode.value),
+
-                relations:
+                tags: this._tagInputNode ?
-                    misc.splitByWhitespace(this._relationsInputNode.value),
+                    misc.splitByWhitespace(this._tagInputNode.value) :
+                    undefined,
+
+                relations: this._relationsInputNode ?
+                    misc.splitByWhitespace(this._relationsInputNode.value) :
+                    undefined,
             },
         }));
     }

client/js/models/post.js

@@ -212,10 +212,10 @@ class Post extends events.EventTarget {
             _canvasHeight:  response.canvasHeight,
             _fileSize:      response.fileSize,
 
-            _tags:          response.tags,
+            _tags:          response.tags || [],
-            _notes:         response.notes,
+            _notes:         response.notes || [],
             _comments:      CommentList.fromResponse(response.comments || []),
-            _relations:     response.relations,
+            _relations:     response.relations || [],
 
             _score:         response.score,
             _favoriteCount: response.favoriteCount,

client/js/util/views.js

@@ -59,6 +59,7 @@ function makeRadio(options) {
             value: options.value,
             type: 'radio',
             checked: options.selectedValue === options.value,
+            disabled: options.readonly,
             required: options.required,
         }) +
     _makeLabel(options, {class: 'radio'});
@@ -75,6 +76,7 @@ function makeCheckbox(options) {
             type: 'checkbox',
             checked: options.checked !== undefined ?
                 options.checked : false,
+            disabled: options.readonly,
             required: options.required,
         }) +
     _makeLabel(options, {class: 'checkbox'});