I lost a little time because I didn't have curl installed (!) on a VPS with only Drupal sites. Main problem was that PHP / Apache didn't log anything about the error 500 I was getting when trying to enable a vocabulary. Turns out the vocabularies are fetched by curl. Maybe documenting this would help someone else?

As a side note: Any particular reason for skipping on using drupal_http_request()?

CommentFileSizeAuthor
#4 microdata-add-readme-1738604-4.patch1.15 KBphizes
#2 microdata-cURL-requirement-1738604-2.patch2.26 KBphizes

Comments

Anonymous’s picture

Anonymous (not verified) commented

Marked #1799020: Fatal error when saving Microdata Settings page. as a duplicate.

BTW, thanks for posting this issue. Unfortunately, I'll be focused on D8 for a few months yet, so I probably won't get a chance to properly fix this for a while, but it should definitely be fixed.

There was a reason for skipping on drupal_http_request, but unfortunately I can't remember why.

phizes’s picture

phizes commented
Status: Active » Needs review
StatusFileSize
new microdata-cURL-requirement-1738604-2.patch2.26 KB

The attached patch adds a README file based on the project page which also lists the cURL requirement. It also adds hook_requirements which prevents the module from being enabled when cURL is not available due to the REQUIREMENTS_ERROR severity level. The hook_requirements is the same as it is in the patch in this comment with minimal changes to make it refer to Microdata instead.

I have tested this patch and it works for me.

Is this in line with what you want to do?

Anonymous’s picture

Anonymous (not verified) commented
Title: Document curl requirement: Error 500 on vocabulary import » Create README

Someone else tested drupal_http_request and had no problems, so they posted a patch in #1810126: Remove dependency on curl library in favour of drupal_http_request which I have committed.

That fixes the main problem that this issue was created for. However, the README part of the patch is still useful.

phizes’s picture

phizes commented
StatusFileSize
new microdata-add-readme-1738604-4.patch1.15 KB

That is a better way to solve the issue.

New patch with just the README. Is there anything else I should be including, or altering, in the README?

damiankloip’s picture

damiankloip commented
Status: Needs review » Fixed

See #1810126: Remove dependency on curl library in favour of drupal_http_request, this hopefully removes the need for this issue now. If we need this, we can re open in the future.

damiankloip’s picture

damiankloip commented
Title: Create README » Create README to document curl dependency

Should probably have a slightly better title too :)

phizes’s picture

phizes commented
Title: Create README to document curl dependency » Create README
Status: Fixed » Needs review

The README patch in comment 4 is just to add a README in general. The cURL dependency was removed with the commit in #1810126: Remove dependency on curl library in favour of drupal_http_request with your patch, as such this issue no longer has anything to do with the previous cURL dependency. The above mentioned patch does not mention cURL at all.

damiankloip’s picture

damiankloip commented

sorry, my mistake.