A debug mode has been introduced in http://cgit.drupalcode.org/flickr/commit/?id=85dbc01 to display all passed photo/album arguments and HTTP requests/response objects via the Devel module.
We would like:
- it display the variable name and function name
- additionally the raw data from the Flickr request
- if the connection method 'cURL' or 'open socket' is used for that.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | link_to_Flickr_response_URL-2396889-9.patch | 10.54 KB | lolandese |
| #1 | better_debug_mode-2396889-1.patch | 5.52 KB | lolandese |
Comments
Comment #1
lolandese commentedAttached patch does as requested, plus:
Comment #2
lolandese commentedComment #5
lolandese commentedIdeally we should get a list of arguments and values that could be inserted directly in the Flickr API Explorer (e.g. https://www.flickr.com/services/api/explore/flickr.photos.search) to test if Flickr returns the desired results.
Comment #6
lolandese commentedAdditionally we could mention some suggested actions in the error message the user could take, like:
It means we should construct a URL to the Flickr API Explorer that includes the used method, the API key from the installed Drupal installation and all the used arguments. This link should be saved to the watchdog to retrieve it later.
Note that the above would work independent from the 'debug mode' as it is shown on error messages only.
Comment #7
lolandese commentedComment #8
lolandese commentedAttached patch displays as a warning message a link to the Flickr response page in XML format (best human readable) with the method used, all argument, the connection method and a link to the corresponding API documentation/Explorer.
The settings should be changed to radio buttons like:
Debug output:
- None
- A link to show the Flickr XML response when a request is made
- Additional Devel output
Comment #9
lolandese commented