¿Can you put links in "Successfully pingbacked URLs" and not only the url?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andreashaugstrup’s picture

I don't understand. The "[URL] pingbacked successfully" message does contain a link. Just click on the URL and you're taken to the URL the pingback was sent to.

Carlitus’s picture

FileSize
40.82 KB

In the form edit node there are a fieldset called "Pingbacks" Inside this fieldset there are a list of "Successfully pingbacked URLs". The problem is that there is a plain list of url and not an anchor list.

I have the last pingback (6.10).

andreashaugstrup’s picture

Assigned: Unassigned » andreashaugstrup

I understand what you're talking about now. I'll make a patch shortly.

jonathan1055’s picture

Title: links in "Successfully pingbacked URLs" » Make links active in "Successfully pingbacked URLs"
Status: Active » Needs review
FileSize
586 bytes

This is a good idea. In pingback_form_alter() I changed

        $past_successes_listing[] = $pb->url;

into

        $past_successes_listing[] = l($pb->url,$pb->url);

Attached is a patch which you can use to do this.

Jonathan

jonathan1055’s picture

Sorry, the patch file above had my own // JSS. comment (which I use to find my own changes in contrib modules). I should have removed this before I rolled the patch. Here is is again

[interestingly, I was able to edit the previous comment, but not delete and re-add the attachment, which is why I had to add this new comment]

andreashaugstrup’s picture

Status: Needs review » Fixed

And this went into HEAD also. Thank you. :o)

Status: Fixed » Closed (fixed)

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