Oscommerce Validato w3c ed accessibile wcag, interessa ?

[maury2ma]

ecco l'ultimo fix per evitare i doppioni.
http://code.google.com/p/oswai/source/detail?r=631
noterai una marea di query "commentate" le ho messe per ricordo
senno' mi perdo i pezzi per strada

[lnaimi]

Ho visto che hai inserito 2 fix (immagino che saranno disponibili entrambe nella prossima stable release). Comunque ho preso un abbaglio: per gestire i listino la cosa è più semplice di quanto potessi immaginare. Nell'ADMIN imposto "Numero di prezzi per prodotto" ad un numero da me scelto pari ai listini che voglio pubblicare. Quando inserisco l'articolo inputo tutti i prezzi differenziati dei listini. Poi vado sulla configurazione del gruppo e per ciascuno dico qual è il prezzo di default. Il gioco è fatto. Non ci sono query con articoli doppi e tutto funziona come dovrebbe. Rimane il fatto che per ciascuno prodotto posso assegnare una sola offerta e non la posso differenziare per gruppo. Non ho avuto modo di applicare i tuoi fix, ma appena ho un secondo guardo se vanno in tale direzione. Grazie, comunque sei sempre un GRANDE

[maury2ma]

si vanno in quella direzione.

sei il primo che usa vari prezzi per articolo in maniera "seria", fammi sapere se saltano fuori problemi.

[lnaimi]

Appena ho un secondo provo i tuoi fix. Per ora stò cercando di integrare in OsWai i seguenti moduli:
Create Basket (molto utile per creare un carrello al posto di un cliente)
Order Edit e Create Account (molto utile per creare un ordine al posto di un cliente)
PDF DataSheet Maker (che lo avevi già inserito te ma poi lo hai tolto, molto utile se devo consegnare una brochure sulle caratteristiche del prodotto)
Poi tutto il resto è OK. Questi sono i moduli che nella vecchia versione di OsCommerce che avevo, il mio ex WebMaster aveva implementato.
Grazie

[lnaimi]

Ti segnalo un bug nella classe class.smtp.php.
Se setto il valore di configurazione "E-Mail da" ad un valore del tipo Nome_del_Negozio <indirizzo email> l'email non parte in alcun modo (l'ho provato con più server smtp)
Quindo ho modificato il metodo come segue e così funziona perfettamente:

function mail($from){
if (eregi("<([a-z0-9\._-]+)(@[a-z0-9\.-_]+)(\.{1}[a-z]{2,6})>$", $from))
{
$lfrom = $from;
}
else
{
$lfrom = '<'.$from.'>';
}
if($this->is_connected()
//AND $this->send_data('MAIL FROM:<'.$from.'>')
AND $this->send_data('MAIL FROM:'.$lfrom)
AND substr(trim($this->get_data()), 0, 2) === '250' ){

return TRUE;

}else
return FALSE;
}
Spero di esserti stato utile. Ciao

[lnaimi]

Ho trovato un altro bug sull'invio delle email in formato HTML (mi stupisco che altri non li abbiano trovati ).
Quando si inserisce una email copiando l'html da un'altra email, l'editor in dotazione formatta il codice inserendo dei \r\n alle righe formattate. Quando queste email vengono spedite il metodo add_html della classe email converte tutti i \r\n in <br /> sformattando completamente il risultato, specialmente se nella email è presente una tabella con parecchie righe. Ho modificato a questo punto il codice del metodo in questa maniera per poter intervenire in modo mirato:

function add_html($html, $text = NULL, $images_dir = NULL, $noBR = FALSE) {
$this->html = preg_replace('/´/i', "'", $html);
$this->html = preg_replace('/'/', "'", $this->html);
if ($noBR==FALSE) $this->html = tep_convert_linefeeds(array("\r\n", "\n", "\r"), '<br />', $html);
// Replace ASCII CR and LF characters with the configured LF character.
// Needs to be done in two passes to prevent doubling of line spacing.
$this->html_text = tep_convert_linefeeds(array("\r\n", "\n", "\r"), "xxCRLFxx", $text);
$this->html_text = tep_convert_linefeeds("xxCRLFxx", $this->lf, $this->html_text);

if (isset($images_dir)) $this->find_html_images($images_dir);
}
Così dove è necessario effettuiamo la sostituzione (comportamento di default), invece nelle newsletter e nell'email dell'admin è possibile settare a TRUE il valore della variabile $noBR e la sostituzione non avviene avendo un risultato ottimale

[maury2ma]

grazie per le segnalazioni

[maury2ma]

ecco le modifiche al codice
ho aggiornato la classe smtp ed ho introdotto il debug in un file

http://code.google.com/p/oswai/source/detail?r=636
grazie per le ottime correzioni

[lnaimi]

Perfetto. Volevo segnalarti un altro bug molto fastidioso. Se disabilito l'obbligo di andare a vedere il carrello ogni volta che aggiungo un nuovo articolo e clicco su acquista su la lista degli articoli in promozione o la lista delle novità, mi dice pagina non trovata, anche se poi l'articolo viene aggiunto al carrello stesso. Se invece obbligo la visione del carrello questo non accade. Poi conosci qualche modulo per la gestione degli RMA. Grazie e sempre disponibilissimo e gentilissimo

[maury2ma]

non lo conoscevo
provvedo quanto prima a fare i dovuti fix

[lnaimi]

Ti segnalo un altro bug sulla classe class.smtp.php. Se utilizzi un server di posta che richiede l'autenticazione per spedire le email, l'invio fallisce. La linea di codice incriminata è
$this->authenticated = EMAIL_SMTP_ACTIVE_PASSWORD;
correggendola a
$this->authenticated = FALSE;
a me funziona perfettamente.

[maury2ma]

ho sbagliato io quando ho importato i parametri all'interno del DB per poterli modificare dall'admin e non intervenire nei file php.
ho associato sia autenticazione che autenticato allo stesso parametro. mentre uno deve essere true e l'altro false.
cambio le variabili nel DB e aggiungo le nuove, in modo da rendere la classe corretta.
grazie per la segnalazione.

inoltre mi sono accorto che alcuni valori booleani vengono passati come stringa... e quindi rompono le scatole
ho aggiornato la classe con i nuovi controlli e nuovi valori

dimmi se saltano fuori nuovi errori.
http://code.google.com/p/oswai/source/detail?r=638

[maury2ma]

per l'altro bug (aggiungi al carrello senza mostrarlo) il lavoro è più articolato:
va in conflitto con uno dei tanti controlli anti-hacker che ho messo, il risultato che un'esecuzione scorretta del codice.
ci sto lavorando.

[maury2ma]

Perfetto. Volevo segnalarti un altro bug molto fastidioso. Se disabilito l'obbligo di andare a vedere il carrello ogni volta che aggiungo un nuovo articolo e clicco su acquista su la lista degli articoli in promozione o la lista delle novità, mi dice pagina non trovata, anche se poi l'articolo viene aggiunto al carrello stesso. Se invece obbligo la visione del carrello questo non accade. Poi conosci qualche modulo per la gestione degli RMA. Grazie e sempre disponibilissimo e gentilissimo

una volta trovato il problema.... la soluzione era ovvia. come sempre.
ecco la semplice modifica:
http://code.google.com/p/oswai/source/detail?r=640

l'unico modulo RMA che conosco è
http://www.oscommerce.com/community/contributions,1136
ma non l'ho mai testato... quindi non saprei se funziona

[maury2ma]

uscita la nuova release
ho risolto alcuni problemi dell'anti-robot che su php 5.3 ogni tanto faceva confusione e prendeva il codice errato dal db
tanti piccoli fix di validazione
un solo modulo nuovo aggiunto, per maggiori info vi invito a visitare il sito del progetto.