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.

Comments

lolandese’s picture

Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new5.52 KB

Attached patch does as requested, plus:

  • set a variable to make Lightbox2 work out-of-the-box if it is installed (unrelated to the issue).
lolandese’s picture

Status: Needs review » Fixed

  • lolandese committed 65066a7 on 7.x-1.x
    Issue #2396889 by lolandese: A better debug mode
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

lolandese’s picture

Status: Closed (fixed) » Active

Ideally 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.

lolandese’s picture

Additionally we could mention some suggested actions in the error message the user could take, like:

Try to reload the page or resave your node or block.

Debug URL that links to the Flickr API explorer to test Flickr's response on the request exactly as it is made by the Flickr module. This URL is saved to the watchdog (admin/reports/dblog). Remember you can filter there to see only the Flickr error messages.

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.

lolandese’s picture

lolandese’s picture

Status: Active » Needs work
StatusFileSize
new1.63 KB

Attached 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

lolandese’s picture

Status: Needs work » Fixed
StatusFileSize
new10.54 KB

  • lolandese committed 673c0ca on 7.x-1.x
    Issue #2396889 by lolandese: Debug info with link to Flickr XML response...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.