cakeplus.add_validation_ruleにバリデーションルール追加
cakeplusを細々と開発していますが、嬉しいことにk1LoWさんにgithub経由でpullリクエストを貰ったので、それをマージしてみました。今回追加したルールは全角のbetween、ひらがなのみ、全角のみかをチェックするの3つです。
今回はgitで一連の操作をしたので、その流れをまとめました。
CakePlusの各機能に関しては、下記の記事の中に書いてあるのでご覧下さい。
http://d.hatena.ne.jp/cakephper/20090909/1252461412
興味を持った方は、ソースコードに使い方などの説明がたくさん書いてあるので、見て頂けると嬉しいです。
http://github.com/ichikaway/cakeplus
マージの前に、とりあえず、現行のコードを初期のバージョンとしてcakeplus-0.1というタグを付けて、push。githubのall tagsという箇所からいつでも取得できるようにします。
http://github.com/ichikaway/cakeplus/tree/cakeplus-0.1
git tag cakeplus-0.1 git push --tags
ブランチを作って、pullリクエストのコードをそこにpullする。
git branch hoge git checkout hoge git pull git://github.com/k1LoW/cakeplus.git
差分は下記で見れます
git diff master hoge
テストケースとか追加してコミット
vi tests/cases/models/behaviors/add_validation_rule.test.php git add tests/cases/models/behaviors/add_validation_rule.test.php git commit -m "add test case"
最後にマスターに移動して、マージして、githubにpush
git checkout master git merge hoge git push git branch -d hoge
git cloneして何かしらCakeplusに機能追加した人は、pullリクエスト下さい(別にメールで連絡でもいいです)。マージしますので。
追記:
今回の追加バリデーションの実装は加賀澤さんという方にして頂きました。ありがとうございます。READMEのContributor欄にお名前を加えさせて頂きました。
追記:
git pullリクエストに対応する方法が下記にまとまってます。
http://help.github.com/pull-requests/#merging_a_pull_request
自分の場合のパターン
$ git branch master-merge $ git checkout master-merge $ git remote add hogehoge git://github.com/hogehoge/cakeplus.git $ git fetch hogehoge $ git merge hogehoge/master //ここで色々と確認、追加、修正作業 $ git co master $ git merge master-merge $ git push origin master $ git branch -d maser-merge $ git remote rm hogehoge