[Risolto] Missing atom:link with rel="self" - patch

Grazie John, lo provo appena passo a Drupal 6.17
Giovanni

Uelà John, sai che ti dico ?
FUNZIONA !!!
Al momento ho applicato la patch in Test, dove ho ancora D.6.15, mentre in produzione ho ancora D.6.16, per cui, lunedì passo a D.6.17 in prod. ed applico la patch.
Ho modificato i due files come da te indicato (common.inc e node.module) ed eseguito l' update.
Lo so che non è il modo migliore per applicare una patch e non mi dire di guardare quà http://drupal.org/patch/apply, troppo complicato per me.
Ho comunque adesso, grazie a te, anche un feed atom senza il modulo Atom ed un feed RSS senza alcun errore.
Giovanni

Patch applicata anche sul mio Drupal 6.17.
Buona lettura.
P.S. nel commento http://www.drupalitalia.org/node/10879#comment-35138 ho erroneamente detto di avere un feed Atom senza il modulo Atom; in test lo avevo proprio perchè avevo installato il modulo Atom che anch' esso non funzionava senza la patch di John.

John posta la patch anche su D.o ;)

PS: ottimo lavoro!

Merci John, molto gentile. L' Italia, purtroppo, al momento, non va proprio così bene (perde 1-0); è opinabile in altri campi.
Giovanni

1-1 . Nè vincitori nè vinti.
Forza Italia !

Infatti stavo vedendo proprio su D.O. il tuo intervento. Lo faccio domani mattina John perchè non sono sul mio P.C.
Ciao.

Fatto!

Aspetta John, sono appena tornato è forse c' è ancora qualcosa che non quadra.

Quote:

FYI, devi togliere le seguenti righe in common.inc
$self_file = (!empty($_SERVER['REQUEST_URI'])) ? $_SERVER['REQUEST_URI'] : '/rss.xml';
$output .= ' \n";

Non so che significa FYI, ma se era rivolto a me, ho ancora la riga indicata in common.inc
Inoltre, nell' intervento http://drupal.org/node/365498#comment-3086480 mi sembra che si parla righe che superano gli 80 caratteri.
Io ho modificato i files common.inc e node.module a mano eliminando i ritorni a capo ed ulteriori spazi.
Fammi sapere se ho fatto qualche casino, anche se, alla mia maniera il feedvalidator funziona.
Giovanni

OK, se FYI ero io, ho eliminato le righe indicate da John sul file common.inc
La validazione del feed RSS è sempre OK.
Ho ancora dubbi per la patch da applicare con i metodi standard avendo io effettuato una modifica manuale.
Giovanni

Chiarissimo. Allora, alla prossima, se non riesci a farti coinvolgere.
Giovanni

@John
Con la nuova versione a Drupal 6.19, i manutentori di D.O. hanno messo le mani anche su node.module con alcune modifiche che non vanno però nella tua direzione. Le modifiche inserite però non soddisfano pienamente la validazione dell' RSS, cosa che è sempre valida invece con la soluzione da te proposta. Personalmente, per ora, non sono ancora in grado di capire le variazioni da loro fatte al node.module; dovrei sapere quale linguaggio è utilizzato e cominciare a studiarlo.
Mi chiedevo se ti va di dargli una occhiata e capire in quale direzione stanno andando. Scusami per la mia ignoranza.
Questa mattina parto per Milano; se riesco seguirò il post da lì.
Giovanni

Test , cosi oggi senza modifiche

Ma la Patch và inserita tal quale nel : modules/node/node.module ?
---------
da
@@ -1665,7 +1665,10 @@ function node_feed($nids = FALSE, $chann
}
........... .........
a
drupal_set_header('Content-Type: application/rss+xml; charset=utf-8');

----------------------------

togliendo poi e anche le righe in common.inc :

$self_file = (!empty($_SERVER['REQUEST_URI'])) ? $_SERVER['REQUEST_URI'] : '/rss.xml';
$output .= ' \n";

bene e grazie : vado a leggere il link e cosi capisco come fare le pac, e questa Patch, (un gradino in più).
Ma drupal org "ha capito" che è una specie di piccolo bug ?
Con wp non credo succede questo.. (succede altro, ah ah)

salvato in CMS/John/feeds-problems

vero (nel mio little : ho capito),
però mi ricorda W3c quando mi diceva : non mettermi "map" metti un link, grazie, oppure metti slash alla fine del nome-link, che aiuta ad elaborare meglio...
sorry allora, correggo: da piccolo bug >>> una svista

@John
Dopo circa una settimana, dall' esilio forzato, ti rispondo.

Quote:

Hai avuto problemi applicare il patch dopo l'upgrade al 6.19?

No, nessun problema, ma devo cambiare il metodo perchè ho sovrascitto il file node.module di D.6.19 con il vecchio del Backup; dicevo che non è un buon metodo perchè, facendo le cose al volo, non ci si accorge se nel file nuovo ci sono state altre modifiche (e penso che ce ne siano state). In pratica, prima di fare l' FTP del vecchio file su quello nuovo, bisognerebbe fare un COMPARE dei due files per verificare se ci sono state altre modifiche e comportarsi di conseguenza; non essendo un esperto in codici, capirai che per me interpretarli non è cosa tanto semplice. Comunque sistemerò le cose al rientro alla base.
@Lorenzo
La modifica, come ti ha detto John, riguarda solo il file node.module.
Ma non dovevamo incontrarci ? Sarò da queste parti ancora per una settimana. Ti invio il mio numero per e-mail; Kipper invece è in vacanza dalle mie parti.
P.S.
Quello che non capisco è perchè quelli di D.O. non applicano la correzione; vogliono farci perdere un pò di tempo ?

@John
Rileggendo questo commento mi domando e dico: "Ma chi sei ?" , immagino la risposta : "Capitan Fracasso !"
Ciao.

Capitan Fracassa .. una Fracassa Utile ..
(il problema è che : è andato al Camp senza neppure farsi la Barba..Bianca..).

ciao giovanni