[RSS-NEWS] errori da sistemare

[Tpierluigi]

@lzuffellato ciao, ci sono dei problemi sulla pagina di RSS del sito.

• index.php il link sulla pagina per le news (che dovrebbero essere i feed RSS) non ha indirizzo corretto (che presuppongo dovrebbe essere "RSS.php"
• RSS.php: i campi che utilizzate nelle select non sono quelli che sono presenti effettivamente nel DB
• RSS.php: c'è ancora una chiamata a mysql_close da togliere, o mettere nel posto giusto..
• insertNewsletter.php: fate un passaggio diretto della variabile GET al db senza fare un minimo di controllo sull'input: se scrivo qualcosa con l'apostrofo lo script va in errore.. e ci siamo salvati da un bell' SQL-injection solo perchè mysql_query fa una sola query alla volta.. :-)

e poi tutto il testo della pagina è ancora con i lorem ipsum eccetera. dovrebbe essere presente il testo e la pagina definitive..

Pierluigi Tonegato [Integrazione&Controllo]

[rentsmdi]

Ciao! ti rispondo subito:

• index.php non è stata creata da noi, ma immagino dal gruppo grafica? perchè, come concordato, il nostro compito era di creare le funzioni che vengono attaccate ai pulsanti/pagine corrispondenti
• connessione al db è lasciato commentato, poichè ancora non sapevamo cosa utilizzare per connetterci al database... e per quanto riguarda i campi abbiamo utilizzato quelli che ha creato per noi il gruppo DB (come indicato nel documento)
• insertNewsletter.php per il get si potrebbe aggiungere qualche funzione per purificare l'imput? per esempio mysqli_real_escape_string();

per il testo ancora da inserire con lorem ipsum etc come dicevo prima non l'abbiamo inserito noi, liberissimi di fare correzioni e aggiunte comunque!

[klleshi]

@LeonardoZuffellato

• Non esiste index.php nella cartella servizi ma index.html
• Visto che voi fate le query sarebbe sensato che guardate anche la connessione al database (a parte: db_name, usename e password che li sistemiamo noi)
• RSS.php: ce una query:

$selezionedati = "SELECT * FROM vRSSFEEDS ORDER BY data"; attributo "data" non esiste nel view vRSSFEEDS ci sono questi attributi "id", "nometab", "news", "snippet"

• insertNewsletter.php: bisogna sistemare il codice per evitare sql-injection.

Il 16/12/2014 11:45, Leonardo Zuffellato ha scritto:

Ciao! ti rispondo subito:

• index.php non è stata creata da noi, ma immagino dal gruppo grafica? perchè, come concordato, il nostro compito era di creare le funzioni che vengono attaccate ai pulsanti/pagine corrispondenti
• connessione al db è lasciato commentato, poichè ancora non sapevamo cosa utilizzare per connetterci al database... e per quanto riguarda i campi abbiamo utilizzato quelli che ha creato per noi il gruppo DB (come indicato nel documento)
• insertNewsletter.php per il get si potrebbe aggiungere qualche funzione per purificare l'imput? per esempio |mysqli_real_escape_string();|

per il testo ancora da inserire con lorem ipsum etc come dicevo prima non l'abbiamo inserito noi, liberissimi di fare correzioni e aggiunte comunque!

— Reply to this email directly or view it on GitHub https://github.com/InformaticaSaraLei/ProgettoSito/issues/54#issuecomment-67141397.

[vpagini]

@Tpierluigi @klleshi @lzuffellato

• Noi abbiamo risposto a quanto chiesto da Tpierluigi. Non esiste la pagina index.php ma solo index.html che è stata aggiunta @fundor333 e @andreafrison e non è competenza nostra.
• RSS.php nel codice SQL v2.1 (http://blogs.unive.it/groups/software_engineering_2012/wiki/08fd1/GenerationX.html) c'è il codice "creazione viste" che contiene la view creata per noi. Nella vista è presente "data" (DATA_INSERIMENTO AS data), andate a leggere il codice se ci sono problemi.
• la connessione al db la sistemiamo domani.
• insertNewsletter.php come detto da lzuffellato, abbiamo proposto di sistemare con una funzione come mysqli_real_escape_string(); In ogni caso, se non vado errata, anche il form di bootstrap per inserire le email nel database fa un controllo sulla correttezza dell'input.

[Tpierluigi]

@lzuffellato @vpagini

• insertnewsletter.php sono d'accordo, procedete pure. Non fidatevi solo del form di bootstrap...
• RSS.php ok, l'avete messo sulla pagina del blog, ma.. come fanno quelli del gruppo database (vedi @TBuc e gli altri) a sapere i campi corretti? vi siete fasati sulla struttura di questa tabella?

poi altre due cosette:

• RSS.php: non si possono gestire meglio le eccezioni? se ad esempio ho un errore dal DB all'interno del ciclo che tira fuori le news non sarebbe opportuno gestirlo e controllare l'output?
• RSS.php: www.nomesito.it lo si può tirar fuori come parametro?

e un'altra cosa al gruppo struttura e grafica: ( @fundor333 @andreafrison )

• il gruppo RSS ha fatto RSS.php, che tira fuori le news in formato XML, il quale è stato linkato sotto sia sul footer che sulla pagina index. dal loro documento si parla solo di sottoscrizione al servizio, ma la index.html, che mi dice: "leggi le news!" allora è sbagliata e il titolo dovrebbe essere qualcosa tipo "iscriviti al feed delle notizie", giusto? se è cosi' allora manca un pezzo perchè il modulo news consultabili dal sito dove sta? chi lo deve fare?
• sul menu principale in home c'è un riferimento a 404.html che non ci deve stare...

Pierluigi Tonegato [integrazione&controllo]

[fundor333]

@Tpierluigi

• Sistemato Link news
• Non ho idea per le NEWS CONSULTABILI DA SITO chi le realizzi
• Il riferimento 404.html è presente perchè NESSUNO ci ha dato il link per il forum

[Tpierluigi]

chiuso, in quanto penso ormai reso obsoleto dallo sviluppo del progetto. Qualora servisse, riaprite pure