Commit e1f79d43 authored by unknown's avatar unknown
Browse files

Merge bk@192.168.21.1:mysql-5.0-opt 

into  mysql.com:/home/hf/work/16813/my50-16813


sql/table.cc:
  Auto merged
mysql-test/r/view.result:
  merging
mysql-test/t/view.test:
  merging
parents f66945aa 5389cc16

mysql-test/r/view.result

+10 −0
Original line number Diff line number Diff line
@@ -2956,4 +2956,14 @@ View Create View 
v1	CREATE ALGORITHM=MERGE DEFINER=`root`@`localhost` SQL SECURITY DEFINER VIEW `v1` AS select `t1`.`pk` AS `pk` from (`t1` join `t2` on(((`t2`.`fk` = `t1`.`pk`) and (`t2`.`ver` = (select max(`t`.`ver`) AS `MAX(t.ver)` from `t2` `t` where (`t`.`org` = `t2`.`org`))))))

DROP VIEW v1;

DROP TABLE t1, t2;

CREATE TABLE t1(id INT UNSIGNED NOT NULL AUTO_INCREMENT PRIMARY KEY, val INT UNSIGNED NOT NULL);

CREATE VIEW v1 AS SELECT id, val FROM t1 WHERE val >= 1 AND val <= 5 WITH CHECK OPTION;

INSERT INTO v1 (val) VALUES (2);

INSERT INTO v1 (val) VALUES (4);

INSERT INTO v1 (val) VALUES (6);

ERROR HY000: CHECK OPTION failed 'test.v1'

UPDATE v1 SET val=6 WHERE id=2;

ERROR HY000: CHECK OPTION failed 'test.v1'

DROP VIEW v1;

DROP TABLE t1;

End of 5.0 tests.

mysql-test/t/view.test

+15 −0
Original line number Diff line number Diff line
@@ -2879,4 +2879,19 @@ SHOW CREATE VIEW v1; 


DROP VIEW v1;

DROP TABLE t1, t2;



#

# Bug #16813 (WITH CHECK OPTION doesn't work with UPDATE)

#

CREATE TABLE t1(id INT UNSIGNED NOT NULL AUTO_INCREMENT PRIMARY KEY, val INT UNSIGNED NOT NULL);

CREATE VIEW v1 AS SELECT id, val FROM t1 WHERE val >= 1 AND val <= 5 WITH CHECK OPTION;

INSERT INTO v1 (val) VALUES (2);

INSERT INTO v1 (val) VALUES (4);

-- error 1369

INSERT INTO v1 (val) VALUES (6);

-- error 1369

UPDATE v1 SET val=6 WHERE id=2;

DROP VIEW v1;

DROP TABLE t1;



--echo End of 5.0 tests.

sql/table.cc

+3 −2
Original line number Diff line number Diff line
@@ -1963,12 +1963,13 @@ bool st_table_list::prep_where(THD *thd, Item **conds, 
            this expression will not be moved to WHERE condition (i.e. will

            be clean correctly for PS/SP)

          */

          tbl->on_expr= and_conds(tbl->on_expr, where);

          tbl->on_expr= and_conds(tbl->on_expr,

                                  where->copy_andor_structure(thd));

          break;

        }

      }

      if (tbl == 0)

        *conds= and_conds(*conds, where);

        *conds= and_conds(*conds, where->copy_andor_structure(thd));

      if (arena)

        thd->restore_active_arena(arena, &backup);

      where_processed= TRUE;