search

mitsukarenai / Projet-Autoblog

Création, gestion et échange d'autoblogs (version 0.3)
Creative Commons Zero v1.0 Universal
48 stars 16 forks source link

Bug XSAF Media #37

Closed ArthurHoaro closed 11 years ago

ArthurHoaro commented 11 years ago

Ce n'est pas encore tout à fait au point. L'ajout automatique se fait bien, mais l'import des medias ne fonctionne pas toujours.

Exemple : http://streisand.hoa.ro//autoblogs/lycracacollecanalblogcom_b01f1fc08e8a6d0dcd1098a6f670af0a2eeadb1b/

(probablement importé de puis la ferme de @mitsukarenai)

mitsukarenai commented 11 years ago

array_shift parameter 1 expected array, NULL given. Mmmh.. Le JSON d'export media semble bon sur mon autoblog, je pense plutôt à un "mauvais traitement de la source" par l'import. Il a récupéré des médias, ou pas du tout ?

ArthurHoaro commented 11 years ago

Ok j'ai pu accéder aux fichier, il y a effectivement un problème sympa.

Contenu du fichier import.json : 

<!DOCTYPE html>
<html lang="en" dir="ltr">
<head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
    <title>Le Lycra ça colle !</title>
    <link rel="canonical" href="http://lycracacolle.canalblog.com/">
    <link rel="alternate" type="application/atom+xml" title="Flux ATOM" href="?feed">
    <style type="text/css" media="screen,projection">
        * { margin: 0; padding: 0; }
    body { font-family:sans-serif; background-color: #efefef; padding: 1%; color: #333; }
    img { max-width: 100%; height: auto; }
        a { text-decoration: none; color: #000;font-weight:bold; } 
   .header a { text-decoration: none; color: #000;font-weight:bold; }
    .header { text-align:center; padding: 30px 3%; max-width:70em;margin:0 auto; }
        .article .title { margin-bottom: 1em; }
    .article .title h2 a:hover { color:#403976; }
        .article h4 { font-weight: normal; font-size: small; color: #666; }
        .article .source a { color: #666; }
        .searchForm { float:right; }
        .searchForm input { }
    .pagination {  background-color:white;padding: 12px 10px 12px 10px;border:1px solid #aaa;max-width:70em;margin:1em auto;box-shadow:0px 5px 7px #aaa; }
    .pagination b { font-size: 1.2em; color: #333; }
    .pagination a { color:#000; margin: 0 0.5em; }
    .pagination a:hover { color:#333; }
    .footer a { color:#000; }
    .footer a:hover { color:#333; }
    .content ul, .content ol { margin-left: 2em; }
    .content h1, .content h2, .content h3, .content h4, .content h5, .content h6,
        .content ul, .content ol, .content p, .content object, .content div, .content blockquote,
        .content dl, .content pre { margin-bottom: 0.8em; }
    .content pre, .content blockquote { background: #ddd; border: 1px solid #999; padding: 0.2em; max-width: 100%; overflow: auto; }
    .content h1 { font-size: 1.5em; }
    .content h2 { font-size: 1.4em;color:#000; }
    .result h3 a { color: darkblue; text-decoration: none; text-shadow: 1px 1px 1px #fff; }
    #error { position: fixed; top: 0; left: 0; right: 0; padding: 1%; background: #fff; border-bottom: 2px solid red; color: darkred; }
.header h1 a { color: #333;font-size:40pt;text-shadow: #ccc 0px 5px 5px;text-transform:uppercase; }
    .article .title h2 { margin: 0; color:#333; text-shadow: 1px 1px 1px #fff; }
    .article .title h2 a { color:#000; text-decoration:none; }
        .article .source { font-size: 0.8em; color: #666; }
    .article { background-color:white;padding: 12px 10px 12px 10px;border:1px solid #aaa;max-width:70em;margin:1em auto;box-shadow:0px 5px 7px #aaa; }
    .footer { text-align:center; font-size: small; color:#333; clear: both; }
    </style>
</head>
<body>
<div class="header">
    <h1><a href="../../" style="font-size:0.8em;">PROJET AUTOBLOG</a></h1>
    <hr>
    <h1><a href="./">Le Lycra ça colle !</a></h1><p>Site original : <a href='http://lycracacolle.canalblog.com/'>Le Lycra ça colle !</a><br><a href="../../">&lArr; retour index</a></p>
    <form method="get" action="http://autoblog.suumitsu.eu/autoblogs/lycracacollecanalblogcom_b01f1fc08e8a6d0dcd1098a6f670af0a2eeadb1b/" class="searchForm">
    <div>
        <input type="text" name="q" value="">
        <input type="submit" value="Recherche">
    </div>
    </form>
</div>

    <div class="article">
        <div class="title">
            <h2>Mise à jour</h2>
        </div>
        <div class="content" id="update">
            Mise à jour de la base de données, veuillez patienter...
        </div>
    </div>
    <div class="article">
        <div class="title">
            <h2><a href="./?2013-07-10-"></a></h2>
            Wednesday 10 July 2013 à 09:32
        </div>
        <div class="content"><p><img src="./media/45867439.88212218.jpg" data-original-source="http://storage.canalblog.com/66/85/1132287/88212218.jpg" alt="SpermeBatmanLIGHT" width="650" height="544" border="0" /></p></div>
        <p class="source">Source : <a href="http://lycracacolle.canalblog.com/archives/2013/07/10/27610030.html">http://lycracacolle.canalblog.com/archives/2013/07/10/27610030.html</a></p>
        <br style="clear: both;" />
    </div>
    <div class="article">
        <div class="title">
            <h2><a href="./?2013-07-09-"></a></h2>
            Tuesday  9 July 2013 à 11:15
        </div>
        <div class="content"><p><img src="./media/284e25ab.88188162.jpg" data-original-source="http://storage.canalblog.com/89/20/1132287/88188162.jpg" alt="SpermeGreenLanternLIGHT" width="650" height="564" border="0" /></p></div>
        <p class="source">Source : <a href="http://lycracacolle.canalblog.com/archives/2013/07/09/27603731.html">http://lycracacolle.canalblog.com/archives/2013/07/09/27603731.html</a></p>
        <br style="clear: both;" />
    </div>
    <div class="article">
        <div class="title">
            <h2><a href="./?"></a></h2>
            Monday  8 July 2013 à 19:58
        </div>
        <div class="content"><p><img src="./media/869919b3.88175134.jpg" data-original-source="http://storage.canalblog.com/57/98/1132287/88175134.jpg" alt="SpermeSpidermanLIGHT" width="650" height="671" border="0" /></p></div>
        <p class="source">Source : <a href="http://lycracacolle.canalblog.com/archives/2013/07/08/27596417.html">http://lycracacolle.canalblog.com/archives/2013/07/08/27596417.html</a></p>
        <br style="clear: both;" />
    </div>
    <div class="article">
        <div class="title">
            <h2><a href="./?2013-07-08-"></a></h2>
            Monday  8 July 2013 à 15:32
        </div>
        <div class="content"><p>&nbsp;</p>
<p><a href="./media/058304be.88167693_o.jpg" data-original-source="http://storage.canalblog.com/79/57/1132287/88167693_o.jpg" target="_blank"><img src="./media/ef6d433f.88167693.jpg" data-original-source="http://storage.canalblog.com/79/57/1132287/88167693.jpg" alt="Hulk3DLIGHT" width="678" height="800" border="0" /></a></p></div>
        <p class="source">Source : <a href="http://lycracacolle.canalblog.com/archives/2013/07/08/27594837.html">http://lycracacolle.canalblog.com/archives/2013/07/08/27594837.html</a></p>
        <br style="clear: both;" />
    </div>
    <div class="article">
        <div class="title">
            <h2><a href="./?2013-07-08-"></a></h2>
            Friday  5 July 2013 à 13:22
        </div>
        <div class="content"><p><img style="display: block; margin-left: auto; margin-right: auto;" src="./media/fe7a1edd.88098671.png" data-original-source="http://storage.canalblog.com/60/86/1132287/88098671.png" alt="batman_robin" width="650" height="498" border="0" /></p></div>
        <p class="source">Source : <a href="http://lycracacolle.canalblog.com/archives/2013/07/05/27574993.html">http://lycracacolle.canalblog.com/archives/2013/07/05/27574993.html</a></p>
        <br style="clear: both;" />
    </div><div class="pagination"><a href="./?1"><b>1</b></a> <a href="./?2">2</a> <a href="./?3">3</a> <a href="./?2">Plus anciens &rarr;</a> </div>
<div class="footer">
    <p>Propulsé par <a href="https://github.com/mitsukarenai/Projet-Autoblog">Projet Autoblog 0.3.0</a> - <a href="?feed">Flux ATOM</a></p>
    <p>Download: <a href="./vvb.ini">configuration</a> (<a href="?opml">OPML</a>)
        - <a href="./articles.db">articles</a><p/>
    <p><a href="./?media">Export fichiers media <sup> JSON</sup></a></p>
</div>
        <script type="text/javascript">
        window.onload = function () {
            document.getElementById("update").innerHTML = "Mise à jour terminée ! <a href=\"#reload\" onclick=\"window.location.reload();\">Cliquez ici pour recharger cette page.</a>";
        };
        </script>
</body>
</html>

Est-ce que ça ne serait pas du au fait que le mustUpdate() bloque l'affichage du JSON des media ?

EDIT : Après un coup d'oeil au code, normalement non...

mitsukarenai commented 11 years ago

Même constat sur TOUS les import.json, alors que les /?media sont bons. xsaf3.php récupère ça lignes 120 et 125 mais sans traiter.

Je vérifie un truc. Je sens que je risque de rager contre une certaine chose, mais je vérifie un file_get_contents à part.

ArthurHoaro commented 11 years ago

Hmm.. Ca marchait quand j'ai testé. J'ai du introduire une régression à un moment à force d'ajuster.

mitsukarenai commented 11 years ago

Nope, PHP récupère bien le bon code c'est pas CloudFlare. Du coup je lorgne sur la ligne 117 de xsaf3.php. Variable changée récemment ?

ArthurHoaro commented 11 years ago

Possible : https://github.com/mitsukarenai/Projet-Autoblog/blame/master/xsaf3.php#L117

Je ne peux pas faire mieux avant ce soir. :s

mitsukarenai commented 11 years ago

(NOTA: peut être que xsaf3.php devrait faire une courte validation de données JSON avant d'écrire import.json, je pense par exemple à des erreurs serveurs "500" qui seraient écrites telles quelles)

ArthurHoaro commented 11 years ago

+1