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

カテゴリプラグインが生成するURLにアクセスした際の表示が異なっている(と思います・・自信なし #485

Closed
tamoot opened this issue Jan 31, 2015 · 13 comments
Milestone

Comments

@tamoot
Copy link
Member

@tamoot tamoot commented Jan 31, 2015

最新版 + rack + puma環境です。

カテゴリプラグインの 表示期間の初期状態

カテゴリ表示画面を表示した時の、最初の表示期間を指定します。 

を四半期に設定したあと、category_to_tagプラグインの生成するパラメータが以下となりました。

http://www.tamoot.net/d/?year=2015;month=1Q;category=Ruby

上記にアクセスすると日記トップページが表示されました。

以下のように category=Ruby を前にもってくると、指定した表示期間かつカテゴリの表示となりました。

http://www.tamoot.net/d/?category=Ruby;year=2015;month=1Q

古いtDiaryやcgi環境だと変わるのかどうか、後で確認してみます。

@tdtds
Copy link
Member

@tdtds tdtds commented Feb 1, 2015

報告された状況はこちらでも確認しました。categoryパラメタを最初に持ってくるのがポイントではなくて、パラメタ区切りを「;」から「&」に変えるとちゃんと期待したとおりの動作をしますね。

dispatcher/index_main.rbではそういう誤動作をする振り分けをしていないので、Rackから渡ってくるresuestが「;」区切りに対応してないのかな。さて、どうしたものか。

@tdtds tdtds added the bug label Feb 1, 2015
@tdtds tdtds added this to the 4.1.2 milestone Feb 1, 2015
@Nyoho
Copy link
Member

@Nyoho Nyoho commented Feb 1, 2015

ひょっとしたら rack が 1.6.0 で、
rack/rack@71c6911#diff-345b5336fcb3ce47a50decd2176d79e8R331
を取り込んで、GETのときでも「;」をセパレータとしなくなってしまったことが原因でしょうか。

hiki を rack を使っていて気付いたメモです: hiki rack メモ - https://gist.github.com/Nyoho/6a20c0975cbeeecd6f9b
rack に pull request を出したみたものを、メモにコメントで追記しておきましたが、実際にどうなおすのがよいか、まだ僕はわかっていません。

@tdtds tdtds removed the bug label Feb 2, 2015
@tdtds
Copy link
Member

@tdtds tdtds commented Feb 2, 2015

W3Cのご威光まで含めてのPR、GJだと思います 👍

常識的に考えれば取り込まれると思うので、tDiary的にはrackのバージョンを1.6未満に指定して様子をみる、という感じかなぁ。

tdtds added a commit that referenced this issue Feb 2, 2015
@tdtds
Copy link
Member

@tdtds tdtds commented Feb 2, 2015

とりあえず'~> 1.5.0'としておきました。このissueはこのままopenしておきます。4.1.2リリース時に見直す感じで。

@hsbt
Copy link
Member

@hsbt hsbt commented Feb 2, 2015

とりあえず'~> 1.5.0'としておきました。

👀 了解です。

@tamoot
Copy link
Member Author

@tamoot tamoot commented Feb 2, 2015

bundle install して rack1.5.2がインストールされた後で期待通りの動作となることを確認できました。

http://www.tamoot.net/d/?year=2015;month=1Q;category=tDiary

@machu
Copy link
Member

@machu machu commented Mar 22, 2015

3/23時点ではまだRackへのPRはまだオープン状態。

rack/rack#781

@tdtds
Copy link
Member

@tdtds tdtds commented Mar 23, 2015

応援しといた

@Nyoho
Copy link
Member

@Nyoho Nyoho commented Mar 23, 2015

応援いただき、ありがとうございます!
さてさてrack側がどう解決されるのか……

@machu machu modified the milestones: 4.1.3, 4.1.2 Mar 28, 2015
@tamoot
Copy link
Member Author

@tamoot tamoot commented May 21, 2015

rack/rack#781 がクローズされたので、 試してみました。四半期表示も問題ありませんでした。

@tdtds
Copy link
Member

@tdtds tdtds commented May 21, 2015

あ、最新のrack gemにこれ入ってるのかな?

@tamoot
Copy link
Member Author

@tamoot tamoot commented May 21, 2015

しまった、、、勘違いしてましたorz 1.6.1に入ってない様子です。

@machu
Copy link
Member

@machu machu commented Jun 29, 2015

rack 1.6.4で取り込まれました。tDiaryでも新しいrackを使うようにします。
rack/rack@a853058

machu added a commit that referenced this issue Jun 29, 2015
machu added a commit that referenced this issue Jun 29, 2015
@machu machu closed this Jun 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants