mirror/php-flarum
1
0
Fork 0
mirror of https://github.com/flarum/core.git synced 2025-08-04 15:37:51 +02:00
Code Issues Releases Wiki Activity

change value of subscribed from false to null in the JSON API (#37)

Browse Source
This commit is contained in:
sl-kr
2021-03-18 12:38:21 -04:00
committed by GitHub
parent c1c47e81aa
commit e783a7603d
4 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
extensions/subscriptions/extend.php
View File

@@ -40,7 +40,7 @@ return [
(new Extend\ApiSerializer(DiscussionSerializer::class))
->attribute('subscription', function (DiscussionSerializer $serializer, Discussion $discussion) {
if ($state = $discussion->state) {
return $state->subscription ?: false;
return $state->subscription;
}
}),

4
extensions/subscriptions/js/src/forum/addSubscriptionControls.js
View File

@@ -10,8 +10,8 @@ export default function addSubscriptionControls() {
if (app.session.user && !(context instanceof DiscussionPage)) {
const states = {
none: {label: app.translator.trans('flarum-subscriptions.forum.discussion_controls.follow_button'), icon: 'fas fa-star', save: 'follow'},
follow: {label: app.translator.trans('flarum-subscriptions.forum.discussion_controls.unfollow_button'), icon: 'far fa-star', save: false},
ignore: {label: app.translator.trans('flarum-subscriptions.forum.discussion_controls.unignore_button'), icon: 'fas fa-eye', save: false}
follow: {label: app.translator.trans('flarum-subscriptions.forum.discussion_controls.unfollow_button'), icon: 'far fa-star', save: null},
ignore: {label: app.translator.trans('flarum-subscriptions.forum.discussion_controls.unignore_button'), icon: 'fas fa-eye', save: null}
};
const subscription = discussion.subscription() || 'none';

6
extensions/subscriptions/js/src/forum/components/SubscriptionMenu.js
View File

@@ -11,7 +11,7 @@ export default class SubscriptionMenu extends Dropdown {
this.options = [
{
subscription: false,
subscription: null,
icon: 'far fa-star',
label: app.translator.trans('flarum-subscriptions.forum.sub_controls.not_following_button'),
description: app.translator.trans('flarum-subscriptions.forum.sub_controls.not_following_text')
@@ -64,11 +64,11 @@ export default class SubscriptionMenu extends Dropdown {
const buttonAttrs = {
className: 'Button SubscriptionMenu-button ' + buttonClass,
icon: buttonIcon,
onclick: this.saveSubscription.bind(this, discussion, ['follow', 'ignore'].indexOf(subscription) !== -1 ? false : 'follow'),
onclick: this.saveSubscription.bind(this, discussion, ['follow', 'ignore'].indexOf(subscription) !== -1 ? null : 'follow'),
title: title
};
if ((notifyEmail || notifyAlert) && subscription === false) {
if ((notifyEmail || notifyAlert) && subscription === null) {
buttonAttrs.oncreate = buttonAttrs.onupdate = vnode => {
$(vnode.dom).tooltip({
container: '.SubscriptionMenu',

2
extensions/subscriptions/src/Listener/SaveSubscriptionToDatabase.php
View File

@@ -18,7 +18,7 @@ class SaveSubscriptionToDatabase
$discussion = $event->discussion;
$data = $event->data;
if (isset($data['attributes']['subscription'])) {
if (array_key_exists('subscription', $data['attributes'])) {
$actor = $event->actor;
$subscription = $data['attributes']['subscription'];