ansaurus

Question

Answer 1

+1 A:

Try changing

 $.each(data, function(){
    $("#sNews ul.tweets").append("<li>" + replaceURLWithHTMLLinks(data.text) + "</li>");

to

$.each(data, function(post, val){
   $("#sNews ul.tweets").append("<li>" + replaceURLWithHTMLLinks(val.text) + "</li>");

Edit:

I did change one other thing but forgot to add it to the answer.

callbackParameter: "jsoncallback",

should be

callbackParameter: "callback",

Just for completeness here is a test page I did that works:

<html>
<head>
   <title>test</title>
   <script type="text/javascript" src="http://ajax.googleapis.com/ajax/libs/jquery/1.3.2/jquery.min.js"&gt;&lt;/script&gt;
   <script type="text/javascript" src="jquery.jsonp-1.1.0.min.js"></script>
   <script type="text/javascript">
      $(document).ready( function() {
         var jsonTwitterFeed = "http://twitter.com/statuses/user_timeline/dougw.json?count=3";

         $.jsonp({
            url: jsonTwitterFeed,
            data: {},
            dataType: "jsonp",
            callbackParameter: "callback",
            timeout: 5000,
            success: function(data){
               $.each(data, function(key, val){
                  $("#sNews ul.tweets").append("<li>" + replaceURLWithHTMLLinks(val.text) + "</li>");                
             });            
         },
         error: function(XHR, textStatus, errorThrown){
             alert("ERREUR: " + textStatus);
             alert("ERREUR: " + errorThrown);
         }
     });
         function replaceURLWithHTMLLinks(text) {
            var exp = /(\b(https?|ftp|file):\/\/[-A-Z0-9+&@#\/%?=~_|!:,.;]*[-A-Z0-9+&@#\/%=~_|])/ig;
            return text.replace(exp,"<a href='$1'>$1</a>");
         }
      });
   </script>
</head>
<body>
   <div id="sNews">
      <ul class="tweets">
      </ul>
   </div>
</body>

beggs 2009-10-27 09:28:28

Does not change anything. I think the success code does not execute at all.

Sergio del Amo 2009-10-27 09:33:37

@Sergio, sorry I forgot one of the changes I made to make it work... see edit in answer. \Cheers

beggs 2009-10-27 14:02:29

Answer 2

A:

I tried with the following code, and got the success function to fire correctly.

$.getJSON(jsonTwitterFeed, {
    success: function(data){
        $.each(data, function(){
            $("#sNews ul.tweets").append("<li>" + replaceURLWithHTMLLinks(data.text) + "</li>");                
        });            

    },
    error: function(XHR, textStatus, errorThrown){
        alert("ERREUR: " + textStatus);
        alert("ERREUR: " + errorThrown);
    }
});

Joel L 2009-10-27 09:45:01

I get the next error with your suggestion: object is undefinedscript.appendChild(document.createTextNo...(object[name],args)===false)break;}else\n

Sergio del Amo 2009-10-27 09:50:36

Answer 3

+1 A:

I don't know if this is entirely relevant but you might have been missing the callback for JSONP in your feed URL unless it's set by default:

var jsonTwitterFeed = "http://twitter.com/statuses/user_timeline/softamo.json?count=3&amp;callback=?";

Also, your jquery AJAX call can simply use the default AJAX and not JSONP. Try this and let us know:

$.ajax({
        url: jsonTwitterFeed,
        data: {},
        dataType: "jsonp",
        callbackParameter: "jsoncallback",
        timeout: 5000,
        success: function(data){
            var str = '';
            for(var i = 0; i < data.length; i++) { 
                str += '<li>' + replaceURLWithHTMLLinks(data[i].text) + '</li>';
            }
            $("#sNews ul.tweets").append(str);
        },
        error: function(XHR, textStatus, errorThrown){
            alert("ERREUR: " + textStatus);
            alert("ERREUR: " + errorThrown);
        }
    });

function replaceURLWithHTMLLinks(text) {
    var exp = /(\b(https?|ftp|file):\/\/[-A-Z0-9+&@#\/%?=~_|!:,.;]*[-A-Z0-9+&@#\/%=~_|])/ig;
    return text.replace(exp,"<a href='$1'>$1</a>");
}

cballou 2009-10-27 09:52:15

i < data.length; i++) {$("#sNews ul.tweets").append("<li>" + replaceURLWithHTMLLinks(data[i].text) + "</li>");}

Sergio del Amo 2009-10-27 10:19:04

@sergio, code updated and I also made a speed improvement to your loop. It avoids multiple DOM lookups until it's absolutely necessary to append the string after all tweets were found and the HTML has been generated.

cballou 2009-10-27 10:57:48

ansaurus

tags:

views:

answers:

How to get Twitter status with JQuery

related questions