Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Not sure what's wrong but I guess the json interface is currently disabled.
drush iq-apply-patch 1762204
is broken currently.
Comment | File | Size | Author |
---|---|---|---|
#8 | drush-iq-2125121-8.patch | 5.11 KB | clemens.tolboom |
#6 | project-issue-2125253.json_.txt | 3.54 KB | clemens.tolboom |
#5 | drush-iq-2125121-5.patch | 1.61 KB | clemens.tolboom |
#1 | drush-iq-2125121-1.patch | 767 bytes | clemens.tolboom |
Comments
Comment #1
clemens.tolboomComment #2
clemens.tolboomIn a dreditor commit https://github.com/dreditor/dreditor/commit/f25b214920f084d28ea67328963b...
Comment #3
helmo CreditAttribution: helmo commentedThe json output was left in the D7 upgrade... #1710850: Deploy RestWS for D7 project issue JSON
Comment #4
clemens.tolboomThis text could probably better link to the correct issue like https://drupal.org/node/1969466 aka #1969466: Fix GET /node/1 with page caching
Comment #5
clemens.tolboom@helmo and @clemens.tolboom are now testing on
We needed to apply Entity #1330086: wrapper throws exceptions for properties that are declared in the info.
Comment #6
clemens.tolboomAttached json file for analysis of the new structure.
Comment #7
clemens.tolboomThis needs documentation as we MUST request for node/1.json as _drush_download_file has no options for adding a header field like "Accept: application/json"
No need to log this.
No need to log this.
This should be a fatal.
Comment #8
clemens.tolboomAttached patch fixes some problems encountered while testing:
- test site runs http instead of hard coded https
- My guess is we need unit tests for the received json data so I split <?php function drush_iq_download_info() by adding <?php function _drush_iq_prepare_json()?
See #2127995: Make testsuite run smoother
Comment #9
greg.1.anderson CreditAttribution: greg.1.anderson commentedThanks for working on this. I guess for now we need to test per https://drupal.org/comment/8145123#comment-8145123?
Will look at this soon.
Comment #10
greg.1.anderson CreditAttribution: greg.1.anderson commentedCommitted #8 with some minor adjustments. Added --issue-site to help, so --strict=0 no longer necessary.
Comment #11
greg.1.anderson CreditAttribution: greg.1.anderson commentedSo, to recap: drush_iq is not yet updated to work with drupal.org, and it appears that drupal.org is not yet emitting json data yet anyway. To test drush_iq, though, you may use the test site, since it -has- been updated to publish json at node/1.json.
To test:
The username and password are both "drupal" (I didn't fill it in because I didn't want to make a valid URL for crawlers to follow...)
Comment #12
greg.1.anderson CreditAttribution: greg.1.anderson commentedContinuation is at #2127965: Fix for the new json structure on d.o D7
Comment #13
clemens.tolboomThanks :)