ok, ma quindi il codice non dovrebbe essere così:

if (ModelState.IsValid)
{
var oldAlbum = db.Albums.Single(m => m.albumID == album.albumID);

// aggiorno l'album passato
//album.thumb = oldAlbum.thumb;
//album.nomeThumb = oldAlbum.nomeThumb;

// aggiorno l'album proveniente dal db
oldAlbum.thumb = album.thumb;
oldAlbum.nomeThumb = album.nomeThumb;


db.SaveChanges();

return RedirectToAction("Index");
}
ViewBag.genreID = new SelectList(db.Genres, "genreID", "description", album.genreID);
return View(album);

con la tua versione è corretto che non vengano effettuate modifiche nel db, perchè aggiorni l'oggetto passato come argomento e non quello proveniente da db

---

Modificato da historyX il 13 marzo 2013 14.55 -

quindi, torno alla domanda di prima, la savechanges non è superflua?

per la prima domanda la risposta è: credo di si, è il framework che decide quale operazione effettuare in base alle operazioni effettuate sull'oggetto. per intenderci è un po come la commit eseguita nel db => se ci sono state modifiche fino a quel momento nel contesto le rende permanenti, diversamente non credo faccia nulla.

Nel tuo caso, il comportamento da te descritto corrisponde al codice che hai postato.
Probabilmente stai sbagliando approccio, ma non posso dirti altro non conoscendo la casistica. (e soprattutto, non conoscendo l'mvc)