Is it worth switching to using http://api.drupal.org/api/function/drupal_http_request/6 rather than CURL in the code that connects to the API?

CommentFileSizeAuthor
#1 845906-notifo-drupal_http_request.patch3.27 KBsnufkin

Comments

snufkin’s picture

snufkin commented
Assigned: Unassigned » snufkin
Status: Active » Needs review
StatusFileSize
new 845906-notifo-drupal_http_request.patch3.27 KB

I believe it is better, because its less code, closer to the general Drupal API as well. This patch uses drupal_http_request and restructures the code to a simpler form. Now there is one function that contains the remote call, the old per-function calls (aka "send a notification", "subscribe user" are now creating a simple query array only and calling this one remote call.

Tested both subscripting, unsubscribing and notifications.

snufkin’s picture

snufkin commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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