From:  pear-qa@lists.php.net ("sasezaki@gmail.com")
Date:  28 Nov 2014 20:26:10 Hong Kong Time
Newsgroup:  news.php.net/php.pear.bugs
Subject:  

[PEAR-BUG] Bug #20451 [Com]: new holiday 'mountain day' is not covered yet.

NNTP-Posting-Host:  null

Edit report at https://pear.php.net/bugs/bug.php?id=20451&edit=1

 ID:               20451
 Comment by:       sasezaki@gmail.com
 Reported By:      sasezaki at gmail dot com
 Summary:          new holiday 'mountain day' is not covered yet.
 Status:           Open
 Type:             Bug
 Package:          Date_Holidays_Japan
 Package Version:  0.1.2
 PHP Version:      Irrelevant
 Roadmap Versions: 
 New Comment:

I made a patch & pull request at github.
https://github.com/pear/Date_Holidays/pull/5


Previous Comments:
------------------------------------------------------------------------

[2014-11-28 12:20:24] sasezaki

-Summary: new holiday 'mountain day'(???) is not covered yet.
+Summary: new holiday 'mountain day' is not covered yet.


------------------------------------------------------------------------

[2014-11-28 12:13:58] sasezaki

Description:
------------
Along with Japanese Holiday enactment at 2014, August 11 should be
judged 
as Holiday.

ref. http://en.wikipedia.org/wiki/Mountain_Day
http://www.shugiin.go.jp/internet/itdb_annai.nsf/html/statics/housei/html/h-
shuhou186.html (The House of Representatives of Japan Information)

Test script:
---------------
 $japan = Date_Holidays::factory('Japan');
 var_dump($obj->isHoliday('2016-08-11'));


Expected result:
----------------
bool(true)


Actual result:
--------------
bool(false)

------------------------------------------------------------------------


-- 
Edit this bug report at https://pear.php.net/bugs/bug.php?id=20451&edit=1