Merge branch 'master' into remove-union

packages/blog/templates/blog_detail.html

@@ -30,13 +30,7 @@
       {% endif %}
         <li>{{ _('{0} views').format(ddoc.views) }}</li>
       {% if handler.user.hasPriv(PRIV.PRIV_USER_PROFILE) %}
-        <li><form class="form--inline" method="post">
-          <input type="hidden" name="operation" value="{% if not dsdoc['star'] %}star{% else %}unstar{% endif %}">
-          <button class="star{% if dsdoc['star'] %} activated{% endif %}" type="submit">
-            <span class="starred--hide"><span class="icon icon-star--outline"></span> {{ _('Star') }}</span>
-            <span class="starred--show"><span class="icon icon-star"></span> {{ _('Unstar') }}</span>
-          </button>
-        </form></li>
+        <li data-star="{{ dsdoc['star'] }}"></li>
       {% endif %}
         <li><a href="{{ url('wiki_help', anchor='contact') }}">
           <span class="icon icon-warning"></span> {{ _('Report') }}

packages/hydrooj/src/handler/discussion.ts

@@ -338,15 +338,10 @@ class DiscussionDetailHandler extends DiscussionHandler {
     }
 
     @param('did', Types.ObjectId)
-    async postStar(domainId: string, did: ObjectId) {
-        await discussion.setStar(domainId, did, this.user._id, true);
-        this.back({ star: true });
-    }
-
-    @param('did', Types.ObjectId)
-    async postUnstar(domainId: string, did: ObjectId) {
-        await discussion.setStar(domainId, did, this.user._id, false);
-        this.back({ star: false });
+    @param('star', Types.Boolean)
+    async postStar(domainId: string, did: ObjectId, star = false) {
+        await discussion.setStar(domainId, did, this.user._id, star);
+        this.back({ star });
     }
 }
 

packages/hydrooj/src/handler/home.ts

@@ -352,28 +352,43 @@ class HomeSecurityHandler extends Handler {
 }
 
 function set(s: Setting, key: string, value: any) {
-    if (s) {
-        if (s.family === 'setting_storage') return undefined;
-        if (s.flag & setting.FLAG_DISABLED) return undefined;
-        if ((s.flag & setting.FLAG_SECRET) && !value) return undefined;
-        if (s.type === 'boolean') {
-            if (value === 'on') return true;
-            return false;
-        }
-        if (s.type === 'number') {
-            if (!Number.isSafeInteger(+value)) throw new ValidationError(key);
-            return +value;
-        }
-        if (s.subType === 'yaml') {
-            try {
-                yaml.load(value);
-            } catch (e) {
-                throw new ValidationError(key);
-            }
-        }
-        return value;
+    if (!s) return undefined;
+    if (s.family === 'setting_storage') return undefined;
+    if (s.flag & setting.FLAG_DISABLED) return undefined;
+    if ((s.flag & setting.FLAG_SECRET) && !value) return undefined;
+    if (s.type === 'boolean') {
+        if (value === 'on') return true;
+        return false;
     }
-    return undefined;
+    if (s.type === 'number') {
+        if (!Number.isSafeInteger(+value)) throw new ValidationError(key);
+        return +value;
+    }
+    if (s.type === 'float') {
+        if (Number.isNaN(+value)) throw new ValidationError(key);
+        return +value;
+    }
+    if (['json', 'yaml', 'markdown', 'textarea'].includes(s.type)) {
+        if (!Types.Content[1](value)) throw new ValidationError(key);
+    }
+    if (s.type === 'text') {
+        if (!Types.String[1](value)) throw new ValidationError(key);
+    }
+    if (s.subType === 'yaml') {
+        try {
+            yaml.load(value);
+        } catch (e) {
+            throw new ValidationError(key);
+        }
+    }
+    if (s.subType === 'json') {
+        try {
+            JSON.parse(value);
+        } catch (e) {
+            throw new ValidationError(key);
+        }
+    }
+    return value;
 }
 
 class HomeSettingsHandler extends Handler {
@@ -484,15 +499,11 @@ class HomeDomainHandler extends Handler {
     }
 
     @param('id', Types.String)
-    async postStar(domainId: string, id: string) {
-        await user.setById(this.user._id, { pinnedDomains: [...this.user.pinnedDomains, id] });
-        this.back({ star: true });
-    }
-
-    @param('id', Types.String)
-    async postUnstar(domainId: string, id: string) {
-        await user.setById(this.user._id, { pinnedDomains: this.user.pinnedDomains.filter((i) => i !== id) });
-        this.back({ star: false });
+    @param('star', Types.Boolean)
+    async postStar({ }, id: string, star = false) {
+        if (star) await user.setById(this.user._id, { pinnedDomains: [...this.user.pinnedDomains, id] });
+        else user.setById(this.user._id, { pinnedDomains: this.user.pinnedDomains.filter((i) => i !== id) });
+        this.back({ star });
     }
 }
 

packages/ui-default/components/star/star.page.ts

@@ -1,6 +1,6 @@
 import $ from 'jquery';
 import { AutoloadPage } from 'vj/misc/Page';
-import { request, tpl } from 'vj/utils';
+import { i18n, request, tpl } from 'vj/utils';
 
 function setStarButtonState($starButton: JQuery<any>, star: boolean) {
   if (star) {
@@ -14,18 +14,17 @@ const starPage = new AutoloadPage('starPage', () => {
   function render(container) {
     const eles = $(container).find('[data-star]');
     for (const el of eles) {
-      if ($(el).find('.star-form')) continue;
-      const pid = $(el).closest('[data-pid]').data('pid');
-      const star = $(el).data('star') === 'true' ? 1 : 0;
-      if (!pid) continue;
-      $(tpl`<form class="form--inline" action="/d/${UiContext.domain._id}/p/${pid}" method="post">
+      if ($(el).find('.star-form').length) continue;
+      const action = $(el).closest('[data-star-action]').data('star-action');
+      const star = ['true', true].includes($(el).data('star')) ? 1 : 0;
+      $(tpl`<form class="star-form form--inline" action="${action}" method="post">
         <input type="hidden" name="operation" value="star">
         <input type="hidden" name="star" value="${star ? 'false' : 'true'}">
         <button class="star${star ? ' activated' : ''}" type="submit">
-          <span class="starred--hide"><span class="icon icon-star--outline" data-tooltip="{{ _('Star') }}"></span></span>
-          <span class="starred--show"><span class="icon icon-star" data-tooltip="{{ _('Unstar') }}"></span></span>
+          <span class="starred--hide"><span class="icon icon-star--outline" data-tooltip="${i18n('Star')}"></span></span>
+          <span class="starred--show"><span class="icon icon-star" data-tooltip="${i18n('Unstar')}"></span></span>
         </button>
-      </form>`);
+      </form>`).prependTo($(el));
     }
   }
 
@@ -39,9 +38,10 @@ const starPage = new AutoloadPage('starPage', () => {
     const currentState = $button.hasClass('activated');
     const $form = $button.closest('form');
     const $op = $form.find('[name="operation"]');
-    if (!['star', 'unstar'].includes($op.val() as any)) return;
+    const $star = $form.find('[name="star"]');
+    if ($op.val() !== 'star') return;
     ev.preventDefault();
-    $op.val(currentState ? 'unstar' : 'star');
+    $star.val(currentState ? 'false' : 'true');
     setStarButtonState($button, !currentState);
     request
       .post($form.attr('action'), $form)

packages/ui-default/templates/home_domain.html

@@ -32,16 +32,8 @@
           <td class="col--icon">
             <img src="{{ avatarUrl(ddoc.avatar, 32) }}" width="32" height="32">
           </td>
-          <td class="col--name">
-            <form class="form--inline" method="post">
-              <input type="hidden" name="id" value="{{ ddoc['_id'] }}">
               {% set _show = handler.user.pinnedDomains.includes(ddoc._id) %}
-              <input type="hidden" name="operation" value="{{ "unstar" if _show else "star" }}">
-              <button class="star{% if _show %} activated{% endif %}" type="submit">
-                <span class="starred--hide"><span class="icon icon-star--outline" data-tooltip="{{ _('Star') }}"></span></span>
-                <span class="starred--show"><span class="icon icon-star" data-tooltip="{{ _('Unstar') }}"></span></span>
-              </button>
-            </form>
+          <td class="col--name" data-action="?id={{ ddoc['_id'] }}" data-star="{{ _show }}">
             {{ ddoc['name'] }} ({{ ddoc['_id'] }})
           </td>
           <td class="col--role">

packages/ui-default/templates/partials/homepage/starred_problems.html

@@ -6,7 +6,7 @@
     </div>
     <div class="section__body typo no-media col--problem-name">
       {%- for pdoc in pdocs -%}
-        <p data-pid="{{ pdoc.docId }}" data-star="true">
+        <p data-action="{{ url('problem_detail', pid=pdoc.docId) }}" data-star="true">
           {{ problem.render_problem_title(pdoc, show_tags=false, show_invisible_flag=false) }}
         </p>
       {%- endfor -%}

packages/ui-default/templates/partials/problem_list.html

@@ -54,7 +54,7 @@
             <td class="col--status record-status--border edit-mode-hide"></td>
           {% endif %}
         {% endif %}
-        <td class="col--name col--problem-name" {% if handler.user.hasPriv(PRIV.PRIV_USER_PROFILE) %} data-star="{{ psdoc.star }}"{% endif %}>
+        <td class="col--name col--problem-name" data-action="{{ url('problem_detail', docId=pdoc.docId) }}" {% if handler.user.hasPriv(PRIV.PRIV_USER_PROFILE) %} data-star="{{ psdoc.star }}"{% endif %}>
           {{ problem.render_problem_title(pdoc) }}
         </td>
         <td class="col--ac-tried">{{ pdoc.nAccept }} / {{ pdoc.nSubmit }}</td>