paquettg / leaguewrap

League of Legend API wrapper
MIT License
67 stars 28 forks source link

Fix Api::Summoner::infoByName() to check $array variable #61

Closed stu177 closed 9 years ago

stu177 commented 9 years ago

If a request is made which causes the $array variable to be empty, such as a non-existent summoner, it causes a PHP error because the foreach loop does not have a valid array to loop over.

coveralls commented 9 years ago

Coverage Status

Coverage increased (+0.01%) to 93.66% when pulling e4d22a2d662926faf058b193f08f04d0545daa9c on stu177:master into 976df8cc121a6dd70463b9152155fb2357a19755 on paquettg:master.

coveralls commented 9 years ago

Coverage Status

Coverage increased (+0.01%) to 93.66% when pulling e4d22a2d662926faf058b193f08f04d0545daa9c on stu177:master into 976df8cc121a6dd70463b9152155fb2357a19755 on paquettg:master.

stu177 commented 9 years ago

Sweet.