Inconsistency between view::load_row and view::save_row for "serialized" fields

For view fields marked a "serialize" (like "display_options"), there is an inconsistency between the way the data is saved in the database and the way the data is reloaded.

In view::save_row (view.inc, line 1632), field value is saved as "serialize($this->$field)"
But in view::load_row (view.inc, line 1675), field value is reloaded as "unserialize(db_decode_blob($data->$field))"

Luckily, for MySQL, db_decode_blob just returns the passed argument so it does not hurt.

But for other types of databases, it might cause I major problem since Views is unable to read back what it saved in the database.

I hit the problem with SQLite, I did not test it but think it will also be a problem with PostgreSQL.

The fix is fairly simple: replace line 1632 in views.inc with :

        $values[] = db_encode_blob(serialize($this->$field));

Thanks for your attention.