SQL server- Transac INSTEAD od update

[samardj]

al compilar el trigger no me bota error.

al tratar de actualizar un nombre con un valor vacío, me bota el error esperado, osea hasta ahí todo bien.

Pero cuando quiero actualizar el nombre con un valor normal , como por ejemplo 'carlos', me bota el siguiente error

"los valores de las filas actualizados no la convierten en única o alteran varias filas"

Debo aclarar que quiero despejar las dudas para fines didácticos, se que puedo usar un AFTER UPDATE y después un rollback, pero deseo saber por que este triger en particular me bota ese mensaje

Muchas gracias por la colaboracion.

acá el ejemplo.


Creación de la tabla, con un ID autoincremental, el cual deber ser único para cada registro

USE [prueba]
GO

/****** Object: Table [dbo].[empleados] Script Date: 03/02/2012 19:25:10 ******/
SET ANSI_NULLS ON
GO

SET QUOTED_IDENTIFIER ON
GO

SET ANSI_PADDING ON
GO

CREATE TABLE [dbo].[empleados](
[id] [int] IDENTITY(1,1) NOT NULL,
[nombre] [varchar](50) NULL,
[apellido] [varchar](50) NULL,
[sueldo] [numeric](18, 0) NULL
) ON [PRIMARY]

GO

SET ANSI_PADDING OFF
GO




y el triger mencionado.


USE prueba;
GO
IF OBJECT_ID ('gatilloup','TR') IS NOT NULL
DROP TRIGGER gatilloup;
GO

CREATE TRIGGER gatilloup ON prueba.dbo.empleados
INSTEAD OF UPDATE
AS

DECLARE @nombre nvarchar(50), @apellido nvarchar(50), @id int;

IF (SELECT nombre FROM inserted) = ''
BEGIN
RAISERROR ('No se puede actualziar a un nombre vacio.', 16, 1);
RETURN
END

ELSE
BEGIN
UPDATE prueba.dbo.empleados
SET nombre = I.nombre,
apellido = I.apellido
FROM prueba.dbo.empleados E, inserted I
WHERE E.id = I.id
RETURN
END


GO

[iislas]

Primiero, estas asumiendo que en tu tabla inserted hay UN SOLO REGISTRO y esto, no es del todo correcto, cuando se haga una actualizacion masiva.

Segundo, debes de utilizar INNER JOIN para ligar tus tablas y dejar de utilizar codigo antiguo en tu WHERE.

[samardj]

gracias iislas, pero no me has sido de ayuda, no estoy asumiendo que hay un solo registro.

Es que hay un solo registro, siempre habra un solo registro al ejecutar un trigger. y todos sus datos estaran en la tabla inserted,

el WHERE es solo sintaxis, funciona perfectamente igual que el INNER JOIN

[iislas]

¿Que version-edicion de SQL Server manejas?

¿Que pasa si hay una actualizacion masiva?

[samardj]

si hay una actualización masiva por ejemplo de 100 registros, el trigger se ejecutara 100 veces por registro, no se si estas familiarizado con los triggers, asumo que si porque decidiste responder mi pregunta, siempre habrá un @@indentity referente a la inserción o actualizacion.

mi pregunta fue una y nos desviamos del tema... y esta al principio.

[Libras]

el error que te esta marcando


"los valores de las filas actualizados no la convierten en única o alteran varias filas"

indica que en el trigger se esta actualizando mas de un valor

y segun lo que veo el error esta aqui:

Código SQL:

UPDATE prueba.dbo.empleados
SET nombre = I.nombre,
apellido = I.apellido
FROM prueba.dbo.empleados E, inserted I
WHERE E.id = I.id
 

ya que pones que actualice donde empleados se encuentre dentro de inserted yo creo que te faltaria esto:

Código SQL:

UPDATE prueba.dbo.empleados
SET nombre = I.nombre,
apellido = I.apellido
FROM prueba.dbo.empleados E, inserted I
WHERE E.id = I.id AND prueba.dbo.empleados.id=I.id
 

:)

[samardj]

Gracias por responder libras, pero si te das cuenta

E es el alias de prueba.dbo.empleados
I es el alias de Inserted

asi que WHERE E.id = I.id , me deberia referenciar a 1 registro.

igual ya cambie le script para evitar confuciones, pero todavia no me da

Código:

BEGIN
   UPDATE empleados 
   SET empleados.nombre = inserted.nombre,
       empleados.apellido = inserted.apellido
   FROM empleados , inserted 
   WHERE empleados.id_emp = inserted.id_emp
   
END

He hecho otro ejercicio y me funciona perfectamente, en otra DB y otra tabla y es el siguiente

Código:

    BEGIN
		UPDATE ventas SET
		ventas.fk_cliente = inserted.fk_cliente,
		ventas.fk_producto = inserted.fk_producto,
		ventas.fecha = GETDATE()
		FROM
		ventas, inserted
		WHERE ventas.id_ventas = inserted.id_ventas
		
		
	END

El segundo codigo si me pfunciona perfectamente, los dos son INSTEAD OF UPDATE

[samardj]

He aqui el error que obtengo.



estoy actualizando desde el management studio, asi que solo actualizo de a 1 registro.

esta porcion de codigo me deberia referenciar ese registro

WHERE empleados.id_emp = inserted.id_emp

[Libras]

Probaste con la solucion que te mencione??? ya revisaste que tus id's no se repitan?? puedes checar si haces en tu trigger un:


select * from inserted
select * from deleted

en cualquier parte del codigo y ver realmente cuales son los id's que estas mandando al update :) prueba y comentas.

Saludos!

[samardj]

al tratar de poner 2 veces lo mismo, osea.

WHERE E.id = I.id AND prueba.dbo.empleados.id=I.id, me bota error de enlace


al tratar de consultar la tabla inserted dentro del codigo del triger no ejecuta 'consulta' nada.

he creado un SP para que im prima la variable, y desde el trigger capturo el id de inserted y lo mando al SP pero no ejecuta nada

[samardj]

he comprobado el id de otra forma y pongo el codigo

Código PHP:

  BEGIN
    SELECT @id_ = id_emp FROM inserted
        
    RAISERROR ('id = %d .', 16, 1,@id_);
    
    UPDATE empleados 
    SET empleados.nombre = inserted.nombre,
        empleados.apellido = inserted.apellido
    FROM empleados ,inserted 
    WHERE empleados.id_emp = @id_
   
END  
   

He puesto el raise error para que me bote el dato del id del inserted, y ciertamente me imprime el numero de id que estoy intentando actualizar.
Luego borro el raiseerror y procedo a actualizar algun dato y me sigue botando el error que les imprimi 3 mensajes atras

[Libras]

amigo tienes un trigger para hacerlo en un update o en un insert???

CREATE TRIGGER gatilloup ON prueba.dbo.empleados
INSTEAD OF UPDATE

INSTEAD OF UPDATE triggers can be defined on a view or table to replace the standard action of the UPDATE statement. Typically, the INSTEAD OF UPDATE trigger is defined on a view to modify data in one or more base tables.

No necesitas un trigger after insert?? si no te aparece nada en el select * from inserted o select * from deleted entonces algo esta pasando, checate el status de tus triggers con este query:

Código SQL:

SELECT trigger_name = sysobjects.name, trigger_owner = USER_NAME(sysobjects.uid),table_schema = s.name, TABLE_NAME = OBJECT_NAME(parent_obj),
  isupdate = OBJECTPROPERTY( id, 'ExecIsUpdateTrigger'), isdelete = OBJECTPROPERTY( id, 'ExecIsDeleteTrigger'),
  isinsert = OBJECTPROPERTY( id, 'ExecIsInsertTrigger'), isafter = OBJECTPROPERTY( id, 'ExecIsAfterTrigger'),
  isinsteadof = OBJECTPROPERTY( id, 'ExecIsInsteadOfTrigger'),
  [disabled] = OBJECTPROPERTY(id, 'ExecIsTriggerDisabled') 
FROM sysobjects INNER JOIN sysusers ON sysobjects.uid = sysusers.uid
INNER JOIN sys.TABLES t ON sysobjects.parent_obj = t.object_id
INNER JOIN sys.schemas s ON t.schema_id = s.schema_id
WHERE sysobjects.TYPE = 'TR'
 

Saludos!

[samardj]

saludos libras, estoy haciendo un triggerINSTEAD OF UPDATE.

Basicamente lo que entiendo es que si hago un triger AFTER UPDATE, ese triger se ejecuta despues de realizar la accion del update, hago las comprobaciones que tenga que hacer, sobre la tabla inserted , y si no cumple algun requisito personal hago un ROLLBACK TRANSACTION, osea deshago el update.

El INSTEAD OF UPDATE, se ejecuta antes de hacer el update en la tabla, y si pasa los requisitos personales, hago el update sobre la tabla, extrayendo los datos de la tabla inserted. (como lo defini en el mensaje anterior)

asi

el nombre de la tabla es inserted para los trigers, insert y update.

en mi caso estoy ejecuntando un INSTEAD OF UPDATE, hago una comprobacion y luego paso a actualizar desde la tabla inserted.

como tambien mostre en el mensaje anterior estoy imprimiendo ese id de la tabla inserted y corresponde al id del registro que estoy actualizando.

lo mas raro de todo es que, como mostre en un mensaje anterior, hice otro ejemplo en otra tabla, y si me hizo el update aplicando el INSTEAD OF UPDATE

he corrido un script para que me muestre los triggers activos en mi db.

USE prueba
GO

SELECT sys.sysobjects.name AS Tabla,sys.triggers.name AS [Trigger], sys.events.type_desc AS Evento,
case when is_disabled = 0
then
'ACTIVADO'
else
'DESACTIVADO'
END As Estado
FROM sys.triggers INNER JOIN
sys.events ON sys.triggers.object_id = sys.events.object_id INNER JOIN
sys.sysobjects ON sys.triggers.parent_id = sys.sysobjects.id
order by sys.sysobjects.name

y solo tengo un trigger UPDATE y es el que estamos discutiendo.

si deseas probarlo, al principio del hilo esta el script de crear tabla, y el de generar el triger para que lo pongas si quieres en tu maquina.

saludos

[Libras]

tambien esta la tabla deleted para los triggers de update, porque no en lugar de hacer un update para probar pintas un select donde tienes tu update algo asi:

Código SQL:

CREATE TRIGGER gatilloup ON prueba.dbo.empleados
INSTEAD OF UPDATE
AS
 
DECLARE @nombre nvarchar(50), @apellido nvarchar(50), @id INT;
 
IF (SELECT nombre FROM inserted) = ''
BEGIN
RAISERROR ('No se puede actualziar a un nombre vacio.', 16, 1);
RETURN
END
 
ELSE
BEGIN
 
SELECT * FROM prueba.dbo.empleados E, inserted I
WHERE E.id = I.id
RETURN
END
 

y vez que es lo que regresa realmente el query si te regresa muchos registros, uno, ninguno etc :)

Saludos!

[samardj]

mm hola libras mira he hecho otro ejemplo y me funciona perfectamente.

esto me desconcierta, que el siguiente ejemplo si sirve.

Código HTML:

USE funciones_
GO

IF OBJECT_ID('productos_up','TR') IS NOT NULL
DROP TRIGGER productos_up
GO

CREATE TRIGGER productos_up ON productos
INSTEAD OF UPDATE
AS
BEGIN
	IF (SELECT nombre from inserted) = 'cigarrillo'
	BEGIN
		RAISERROR('No se puede insertar ese producto',16,1)
	END
	
	ELSE
	BEGIN
		UPDATE productos SET
		productos.nombre = inserted.nombre,
		productos.precio = inserted.precio,
		productos.tiene_iva = inserted.tiene_iva
		FROM productos,inserted
		WHERE productos.id_producto =  inserted.id_producto 
	END

END
GO

[Libras]

y que te regresa el trigger que te recomende?

[samardj]

wl trugger que em recomendaste no me regresa nada, nunca me ha devuelto datos un trigger, ni siquiera un SELECT 3+4 as suma, entonces la unica forma de comprobar los valores dentro del trigger, por lo menos para mi, es con el raiseerror e imprimir el ID del inserted

[Libras]

Como que un trigger no regresa nada? si yo tengo triggers after insert, after update que si les pongo un select * from inserted o select * from deleted me regresan los registros que se estan manejando, si le pongo un select 4+3 me regresa el resultado tambien...si no te regresa nada cuando pones un select * from inserted o un select * from deleted entonces algo esta pasando con tus triggers o con ese trigger :P, ya intentanste borrarlo y crearlo con un nuevo nombre??