Skip to content

Commit

Permalink
Fix CORE-2985: The new 2.5 feature to alter COMPUTED columns doesn't …
Browse files Browse the repository at this point in the history
…handle dependencies well. Also fix the similar situation for ALTER TRIGGER statement
  • Loading branch information
skidder committed May 2, 2010
1 parent 8023bee commit 5b34865
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/jrd/dfw.epp
Expand Up @@ -4478,11 +4478,14 @@ static bool find_depend_in_dfw(thread_db* tdbb,
break;
}

// Look to see if an object of the desired type is being deleted.
// Look to see if an object of the desired type is being deleted or modified.
// For an object being modified we verify dependencies separately when we parse its BLR.
for (const DeferredWork* work = transaction->tra_deferred_job->work; work; work = work->getNext())
{
if ((work->dfw_type == dfw_type ||
(work->dfw_type == dfw_modify_procedure && dfw_type == dfw_delete_procedure) ||
(work->dfw_type == dfw_modify_field && dfw_type == dfw_delete_global) ||
(work->dfw_type == dfw_modify_trigger && dfw_type == dfw_delete_trigger) ||
(work->dfw_type == dfw_modify_function && dfw_type == dfw_delete_function)) &&
work->dfw_name == object_name && work->dfw_package.isEmpty() &&
(!rel_id || rel_id == work->dfw_id))
Expand Down

0 comments on commit 5b34865

Please sign in to comment.