OpenWeatherMap-PHP-API icon indicating copy to clipboard operation
OpenWeatherMap-PHP-API copied to clipboard

Ideas for future versions

Open cmfcmf opened this issue 8 years ago • 4 comments

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 from City.
  • [ ] 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.

cmfcmf avatar Apr 12 '16 17:04 cmfcmf

Achieve 100% test code coverage?

aseriy avatar Apr 12 '16 18:04 aseriy

Hi @aseriy , the code coverage is very closed to the 100%. Please refer this PR. Thanks.

peter279k avatar Jan 06 '17 21:01 peter279k

The official OpenWeather API service now supports the air pollution service. Is it possible to add this feature in the next version?

Thanks.

peter279k avatar Jan 30 '17 08:01 peter279k

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 :)

cmfcmf avatar Feb 01 '17 20:02 cmfcmf