Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

新着情報の表示件数を変更すると、日数表示に変わってしまう #1253

Closed
onozaki opened this issue Jun 12, 2018 · 8 comments

Comments

@onozaki
Copy link

onozaki commented Jun 12, 2018

新着プラグインの設定を「最新5件を新着とする」にしているのですが、
表示件数を変更しようとブルダウンより件数を選んだところ、
日数表示(3日分)に切り替わってしまい、その後、日数表示の変更しかできなくなりました。
  NetCommons 3.1.8 【新着プラグイン3.1.7】

default
       ↓
default

この状態で設定を見ると、「最新××日分を新着とする」に変わっており、表示項目も最初の設定と違い、すべてチェック☑が入ってしまいました。
default

 【最初の設定】
default

■ NetCommons 3.1.7 【新着プラグイン3.1.7】で同じことを試したところ、
 上記のような現象はなく、件数表示の変更ができました。

@RikaFujiwara
Copy link
Contributor

https://github.com/NetCommons3/NetCommons/blob/master/View/Helper/DisplayNumberHelper.php#L178-L179
ここで件数切り替え後のURLを設定しているところでFrameIdの引数設定が漏れてるからじゃないかなと思うのですが、いかがですかね?

@akagane99
Copy link
Contributor

@schit55555
Copy link

schit55555 commented Dec 28, 2018

あまり細かく検証はしておりませんが、下記のように設定したところできたようです。
他に影響があるかどうかわかりませんが、いかがでしょうか。

app/Plugin/NetCommons/View/Helper/DisplayNumberHelper.php

                 if (Current::read('Block.id')) {
                            $attributes['url'] = NetCommonsUrl::blockUrl($named);
                    } else {
                            #$attributes['url'] = NetCommonsUrl::actionUrlAsArray($named);
                            $attributes['url'] = NetCommonsUrl::actionUrlAsArray(
                                                    array(
                                                            'action' => 'index',
                                                            'block_id' => Current::read('Block.id'),
                                                            'frame_id' => Current::read('Frame.id'),
                                                    )
                            );
                    }

@akagane99
Copy link
Contributor

情報感謝です^^
DisplayNumberHelper.phpという事で、共通部品の修正なんですね。
社内で聞いてみたところ、新着の件数切替や、ブログ、掲示板の件数切替はちゃんと動いたようです。
しかし、新着の「もっと見る」機能が動かなくなったと聞きました。

時間できたら見てみますね。

@schit55555
Copy link

ありがとうございます、お忙しいところすみませんがよろしくお願い致します。

@akagane99
Copy link
Contributor

akagane99 commented Dec 29, 2018

https://github.com/NetCommons3/NetCommons/pull/516/files
テスト中ですが、こんな感じで修正してみました。

しかし、新着の「もっと見る」機能が動かなくなったと聞きました。

こちらは既存バグで、今回の修正とは無関係でした。失礼しました。
#1186

途中経過まで。

@akagane99
Copy link
Contributor

akagane99 commented Dec 31, 2018

修正プログラムをマージして、対応しました。

@akagane99
Copy link
Contributor

修正内容見直して、新着のプログラムを修正するようにしました。
https://github.com/NetCommons3/Topics/pull/113/files#diff-5a273dc9ebacea1b836a2274d9fa22d2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants