bello lungo sto codice.......
comunque io l'ho già visto in qualche altro post, di qualche altro utente disperato nel modificare questa pagina. ma arriva da un CMS od un sito OpenSource? Come si chiama?
Tornando al codice:
innanzi tutto hai due funzioni attaccate ad un evento, e questo non andrebbe bene:
Protected Sub ImgBtnVaiAllaCassa_Click(ByVal sender As Object,
ByVal e As ImageClickEventArgs)
Handles ImgBtnVaiAllaCassa.Click
...
...
Public Sub btnAcquistaControlla(ByVal sender As Object,
ByVal e As ImageClickEventArgs)
Handles ImgBtnVaiAllaCassa.Click
la prima la puoi proprio togliere, non serve a niente.
Dentro la seconda funzione, nel ramo if, hai la seguente riga:
ScriptManager.RegisterStartupScript(Me,
Me.GetType,
"SpedMancante",
"alert('Spedizione mancante!');",
True)
come ti dicevo in precedenza, è perfettamente inutile e scorretta poiché siamo dentro un UpdatePanel.
Poi c'è una cosa che, anche se funziona, non i piace per niente:
CodeFile="Carrello.aspx.vb" Inherits="_Default"
Perché la pagina carrello eredita una classe _Default, ovvero, perché il codebehind della pagina carrello costruisce una classe _Default? Non mi piace per niente e tra l'altro è "scorretto" non solo per i puristi del codice.
Il resto, anche se lunghissimo e non l'ho spulciato a fondo, mi sembra ok.