OpenWeatherMap-PHP-API
OpenWeatherMap-PHP-API copied to clipboard
Ideas for future versions
Cleanup and Minor Features
- [x] Remove deprecated methods and constructor arguments from the OpenWeatherMap class
- [x] Change
° C
back to°C
(The API should not be HTML aware.) - [x] Add better error handling
- [x] Make sure to handle invalid data from OWM properly (#100)
- [ ] #131
- [ ] Remove
population
field fromCity
. - [ ] Remove
$appid
from all methods except constructor - [ ] Split forecast apis into two methods (no more magic depending on the number of requested days)
- [ ] Rename temperature->current to temperature->avg
- [ ] Split up Temperature class into separate class for 16 day forecast
- [ ] Merge getCurrentWeather() and getCurrentWeatherGroup()
Documentation
- [x] Remove Examples and write a proper documentation instead #98
- [ ] Document all the OWM bugs this API fixes
Tests
- [ ] Achieve 100% test code coverage
- [ ] Find a tool to create a mock api server
- [ ] Integrate mock api server
New Features
- [ ] Support more / all OWM apis
- [ ] ~~Support unit conversion #37~~
- [x] Replace "Fetcher" classes by ~~HTTPlug http://httplug.io/~~ PSR 18
- [x] Replace custom caching by PSR-6 implementation http://www.php-fig.org/psr/psr-6/
- [ ] Add PSR\Logger support (especially useful for debugging nasty OWM errors)
You are welcome to propose further (backwards incompatible) changes.
Achieve 100% test code coverage?
Hi @aseriy , the code coverage is very closed to the 100%. Please refer this PR. Thanks.
The official OpenWeather API service now supports the air pollution service. Is it possible to add this feature in the next version?
Thanks.
Is it possible to add this feature in the next version?
It certainly is possible to add this feature. However, I currently do not have the time to do so. I'd gladly accept contributions though :)