Coming from an IRC discussion with merlinofchaos and dawehner, we need to check for the machine name of the view before saving. This is needed and can cause problems when saving exportables to the database. If they already exist, this is not known when saving the view as there is no vid. Patch shortly.

CommentFileSizeAuthor
#2 1527778-2.patch580 bytesdamiankloip
#1 1527778.patch602 bytesdamiankloip

Comments

damiankloip’s picture

StatusFileSize
new602 bytes
damiankloip’s picture

StatusFileSize
new580 bytes

Here is amended patch, only checking if $view->vid is empty.

damiankloip’s picture

Issue summary: View changes

Updated issue summary.

dawehner’s picture

Status: Needs review » Fixed

This patch looks pretty sane.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

Updated issue summary.