Nucleus(JP)フォーラム

NucleusCMS日本語版ユーザーのためのサポートフォーラムです。疑問が生じたらまずは記事検索をご利用ください。

ログインしていません。

#1 2018-12-08 18:55:53

tousei
メンバー
登録日: 2018-12-08
投稿: 4

xmlrpc メソッドの追加実装を contribute したいが方法は?

はじめまして。

私は Windows Live Writer などの xmlrpc クライアントを使って nucleus に投稿していますが、
投稿日時を指定して投稿しても nucleus が日時を受け付けないため、ソースを見たところ、
metaWeblog API の newPost, editPost に日時指定に必要な処理が実装されていませんでした。

そこで、日時指定で投稿した場合に正常に日時を設定できるようコードを追加修正したので、
contribute したいのですが、方法が分かりません。

私のバージョンは 3.71 日本語版ですが、恐らく 3.8RC などでも xmlrpc 関連は変わらずだと
思いますので最新版でも役立てることができると思います。

英語版のフォーラムが本家なのかと思い、 Register しようと試みたのですが、私の e-mail
アドレスにはアクティベーション用の e-mail が送信されてこなかったので、メッセージを
届けることすらできないため、こちらで質問しました。

Mars Edit, Windows Live Writer などの xmlrpc クライアントを使っている方にとっては、
現状では不便だと思うので、 contribute の方法をご教示いただけませんでしょうか。

編集者 tousei (2018-12-08 18:56:38)

オフライン

#2 2018-12-21 10:42:07

yama
Administrator
登録日: 2005-07-07
投稿: 1,269
ウェブサイト

Re: xmlrpc メソッドの追加実装を contribute したいが方法は?

すみません、確認が遅くなりました。以下、簡単に手順を説明しますね。
できればGitの環境をお持ちのほうがいろいろ便利だと思いますが、とりあえずブラウザだけで
できる方法を説明します。

https://github.com/NucleusCMS/NucleusCMS
本家版
https://github.com/NucleusCMS/NucleusCMS-ja
日本版

できれば本家版がいいですが、いずれ廃止予定の日本版のリポジトリを練習代わりに使って
いただくのもよいと思います。

https://github.com/NucleusCMS/NucleusCM … pc.inc.php
編集したいファイルをブラウザで表示して、右上のペンシルアイコンをクリックします。(要ログイン)
修正したソースコードを貼り付けて、
Create a new branch for this commit and start a pull request.
を選んで Commit changes をクリックします。コメントを書いていただく必要がありますが
日本版のほうは日本語で分かるように、メモ程度でいいので何か書いていただけたらと思います。
本家版は英語で書きます。あまり深く考えず、伝わればいいという程度でいいですよ。
start a pull request なので、この方法による投稿は即座にコミットされるわけでなく、
こちらで内容を確認してから採用する流れになります。

オフライン

#3 2018-12-22 15:10:56

tousei
メンバー
登録日: 2018-12-08
投稿: 4

Re: xmlrpc メソッドの追加実装を contribute したいが方法は?

yama さんの発言:

https://github.com/NucleusCMS/NucleusCM … pc.inc.php
編集したいファイルをブラウザで表示して、右上のペンシルアイコンをクリックします。(要ログイン)
修正したソースコードを貼り付けて、
Create a new branch for this commit and start a pull request.
を選んで Commit changes をクリックします。コメントを書いていただく必要がありますが

yama さん、ご教示ありがとうございます。

ご教示の通りにペンシルアイコンをクリックし、コードを修正しましたが、その画面に、「Create a new branch for this commit and start a pull request.」という表示はどこにもありませんでした。

画面の下部には、コメントの入力欄と共に、「Propose file change」というボタンがありました。

画面の上部には、「You’re editing a file in a project you don’t have write access to. Submitting a change to this file will write it to a new branch in your fork tousei-seiso/NucleusCMS, so you can send a pull request.」というメッセージが青い枠付きで表示されていました。

メッセージを読む限りは、私のアカウントには NucleusCMS/NucleusCMS のファイルに書き込む権利はないので、自分で既に fork して試していた tousei-seiso/NucleusCMS に「Propose file change」ボタンから一旦書き込むことで、後で NucleusCMS/NucleusCMS 本体への pull request を発行できるということではないのかな?と思い、「Create a new branch for this commit and start a pull request.」ボタンを探し回るのは諦めて、ひとまず「 Propose file change」ボタンを押して自分の fork した branch に書き込む方法で試行錯誤してみました。

修正したファイルは1つではなく複数なのですが、この方法だと同じ機能追加のための修正に対し、ファイルごとに別々の pull request の発行が必要になってしまいましたが、それでいいんでしょうか?1つの pull request にまとめる方法は分かりませんでした。

間違いなどありましたらご指摘いただければ嬉しいです。

編集者 tousei (2018-12-22 15:12:11)

オフライン

#4 2018-12-22 20:30:52

yama
Administrator
登録日: 2005-07-07
投稿: 1,269
ウェブサイト

Re: xmlrpc メソッドの追加実装を contribute したいが方法は?

なるほど、そうだったかもです。すみません。汗
コミットを見たのですが、どうもいろいろ説明が足りてませんでした。
すみませんが少しお付き合いいただけると助かります。

masterブランチは現在公開中のバージョンで、つまりv3.7系なので、現在開発中の
v3.8系とはかなり構成が違います。なので、何らかの理由があってv3.7系を修正する
必要がある場合はv3.7系であるmasterブランチに対してコミットすればよいのですが、
そうでなければ基本的には開発用の最新ブランチに対してコミットする必要があります。

https://github.com/NucleusCMS/NucleusCMS/pull/86
かなり気合の入ったプルリクみたいで申し訳ないです。すみませんが、もう一度v3.8に対して
作っていただいてよいでしょうか?最初は練習のつもりで簡単なプルリクをひとついただけたら、
それをこちらで監査してすぐにマージしますので、それで感覚をつかめると思います。

https://github.com/NucleusCMS/NucleusCMS/tree/v3.8dev
本家版のこちらのブランチに対する改修が必要です。

https://github.com/NucleusCMS/NucleusCMS-ja
日本版はv3.8がデフォルトになってます。
日本版は今後は積極的には使用を推奨しないので、プルリクはなくてもよいです。
今までの日本版の仕様は、本家版v3.8devで取り込まれています。

オフライン

#5 2018-12-22 23:29:38

tousei
メンバー
登録日: 2018-12-08
投稿: 4

Re: xmlrpc メソッドの追加実装を contribute したいが方法は?

yama さんの発言:

最初は練習のつもりで簡単なプルリクをひとついただけたら、
それをこちらで監査してすぐにマージしますので、それで感覚をつかめると思います。

https://github.com/NucleusCMS/NucleusCMS/tree/v3.8dev
本家版のこちらのブランチに対する改修が必要です。

こちらこそ何度もご教示ありがとうございます。確認してみましたら、確かに 3.8dev ではなく master への PR になってしまっているようでした。

最初は簡単な PR をひとつ・・・ということでしたので、 _categoryList() の修正だけやってみました。
「bug fix of _categoryList() #89」が、そうです。

>tousei-seiso wants to merge 1 commit into NucleusCMS:v3.8dev from tousei-seiso:patch-5
と表示されましたので、 本家 3.8dev への PR に出来たと思いますが、どうでしょうか?

オフライン

#6 2018-12-23 10:32:05

yama
Administrator
登録日: 2005-07-07
投稿: 1,269
ウェブサイト

Re: xmlrpc メソッドの追加実装を contribute したいが方法は?

https://github.com/NucleusCMS/NucleusCMS/pull/89
ありがとうございます、マージしました。

オフライン

#7 2018-12-23 14:19:28

tousei
メンバー
登録日: 2018-12-08
投稿: 4

Re: xmlrpc メソッドの追加実装を contribute したいが方法は?

yama さんの発言:

https://github.com/NucleusCMS/NucleusCMS/pull/89
ありがとうございます、マージしました。

大丈夫だったようでよかったです。おかげさまで PR の基本的なやり方は分かりました。

同様に、 api_metaweblog.inc.php, server.php の2つの PR を上げました。同じ目的の修正ですが、こういう場合に PR を一つにまとめる方法があるかが分からなかったので、個別に PR を立てましたが、まずかったらすみません。

オフライン

Board footer