Bug#822481: jessie-pu: package wmforecast/0.8-1

["Adam D. Barratt" <adam@adam-barratt.org.uk>]

Control: tags -1 + confirmed

On Sun, 2016-04-24 at 17:47 -0400, Doug Torrance wrote:
> In March 2016, Yahoo! changed their weather API in a non-backwards compatible
> way.  This made wmforecast, a Window Maker dockapp which relies on this API
> to display weather information, nonfunctional.
> 
> A new release, version 0.10, was prepared to fix this problem, and additional
> improvements were made in 0.11, now available in stretch.
> 
> However, jessie still has version 0.8.  I have backported the appropriate
> changes from 0.10 and 0.11 into a small patch which makes version 0.8
> functional.  I would like to propose that the jessie wmforecast package be
> updated with these changes.

++	url = wstrdup("https://query.yahooapis.com/v1/public/yql?q=";
++		      "select%20*%20from%20weather.forecast%20where%20woeid");
+ 	if (strcmp(prefs->woeid_or_zip,"w") == 0) {
+-		url = wstrappend(url,"&w=");
++		url = wstrappend(url, "%20%3D%20");
+ 		url = wstrappend(url, prefs->woeid);
+ 	}
+ 	else {
+-		url = wstrappend(url,"&p=");
++		url = wstrappend(url, "%20in%20(select%20woeid%20from%20"
++				 "geo.places(1)%20where%20text%3D%22");

I really do hope the API isn't just literally executing those strings...

Please go ahead.

Regards,

Adam