search

inc2734 / mw-wp-form

69 stars 46 forks source link

郵便番号のバリデーションで、ハイフンがあってもなくても通るように #35

Closed mypacecreator closed 9 years ago

mypacecreator commented 9 years ago

入力欄が2つに分かれててほしくない人向けです。

inc2734 commented 9 years ago

テストがこけているので間違いがある可能性があります。 こんな感じだとどうでしょう？

!preg_match( '/^\d{3}-?\d{4}$/', $value )
mypacecreator commented 9 years ago

そっちのほうがいいですね。中途半端で送っちゃってすみません。 それにclass.tel.phpに揃えて、preg_matchを2つに分けて書いたほうがよかった気がします。いまさらですが^^;;

inc2734 commented 9 years ago

なるほどです。 ( 再度プルリクいただけると超うれしいです…！ )

mypacecreator commented 9 years ago

すみませんひとつ教えてください。 確認したら tests/validation-rules/test-mw-wp-form-validation-rule-zip.php で、ハイフン無し7桁の数字が「郵便番号の形式じゃない」ってことになっているのでfailedになってる、ってことですよね？ この場合は、テストコードの方を書き換えていいんでしょうか。

inc2734 commented 9 years ago

そうですね。下記のコードに書き換えてテストが通れば問題ないです！

    /**
     * @backupStaticAttributes enabled
     */
    public function test_郵便番号の形式ならnull() {
        $this->Data->set( 'text', '000-0000' );
        $this->assertNull( $this->Rule->rule( 'text' ) );
        $this->Data->set( 'text', '0000000' );
        $this->assertNull( $this->Rule->rule( 'text' ) );
    }

    /**
     * @backupStaticAttributes enabled
     */
    public function test_郵便番号の形式以外ならnotnull() {
        $this->Data->set( 'text', 'aaa' );
        $this->assertNotNull( $this->Rule->rule( 'text' ) );
    }
mypacecreator commented 9 years ago

遅くなってすみません！テストルール変更したら大丈夫そうでした^^

inc2734 commented 9 years ago

せっかくプルリクいただいているのに遅くなってすみません！！！！ ちょっと最近いそがしくてなかなかプラグインのメンテの時間がとれないのですが、なるべく早くマージできるようにしたいと思っています。おまたせしてすみません。。