topic Re: Transaction inside an IUpdater in Revit API Forum

Transaction inside an IUpdater

ridaabderrahmane — Fri, 10 Jan 2020 21:17:05 GMT

Hi everyone,

here is my situation i am trying to change the floor above a room when it's boundaries change.

for that i implemented an IUpdater to track the changes of my rooms and look for the floor above it to change it, but to make that possible i used the method Room.IsInRoom(Point) where the Point belongs to the floor.

to get a point that belongs to the floor i had to do a transaction and roll it back to get the model lines of the floor However the plugin throws an exception because we cannot make a transaction inside an Updater.

Is there a way to do things in another way ?

Thanks in advance.

Re: Transaction inside an IUpdater

cwaluga — Fri, 10 Jan 2020 23:08:49 GMT

If you need to use the temporary transaction trick, you could subscribe to the DocumentChanged event instead of using the updater. You will need to use a FilteredElementCollector though to filter for the spaces, since DocumentChanged triggers for anything.

Re: Transaction inside an IUpdater

Sean_Page — Sat, 11 Jan 2020 02:43:19 GMT

You don't need a Transaction in an Updated because it happens during the transaction of the element being updated, IE in a transaction. Just remove the transaction and it should be fine. But then again, you shouldn't even need a transaction if you're just getting a point and checking if it's in a room.

Re: Transaction inside an IUpdater

jlpgy — Sat, 11 Jan 2020 06:07:13 GMT

Hi:

IUpdater.Execute() method is still inside a started Transaction 🙂

Re: Transaction inside an IUpdater

ridaabderrahmane — Sat, 11 Jan 2020 10:44:28 GMT

H8 everyone,

thanjs for your responses,

i tried to use the onchangeddocument method but it doesn't seem to be woorking because it is read only and can t be used to modify the project.

the transaction i am trying to use is not to perform a change in the document. it is just a temporary transaction to get the floor points, i wonder if there is a way to get the floor center without performing that temporary transaction, using the floor bounding box for example ?

thans in

Re: Transaction inside an IUpdater

FAIR59 — Sun, 12 Jan 2020 03:06:16 GMT

As there is an active transaction, you can simply use a subtransaction instead of a transaction.

Re: Transaction inside an IUpdater

ridaabderrahmane — Sun, 12 Jan 2020 11:52:10 GMT

Hi dear,

Thank you very much for your response, i tried to use a subtransaction instead, the updater fails to roll back the subtransaction elthough i checked the subtransaction status and there is no other opened subtransaction in my project.

Kind regards;

Re: Transaction inside an IUpdater

jlpgy — Mon, 13 Jan 2020 03:56:55 GMT

Hi @ridaabderrahmane :

I do think that your Title is misleading everyone.

Let's suppose that you aim is to fetch the geometry center of a <Floor>.

And your personal (unsuccessful solution) is :

Using a <IUpdater>
Start a <SubTransaction> druing auto updating procedure
Do something to fetch some temp model lines
Rollback <SubTransaction>
Do anything else which you REALLY want to do

editing-a-floor-profile.html

Check this post in JT's blog to see if it can help.

And the API class <Floor> methods might help:

GetNormalAtVerticalProjectionPoint

GetVerticalProjectionPoint

Perhaps you are willing to share some codes to show everyone how you were doing and what you were intended to do?

🙂

If there is a way to easily fetch the feature points of a <Floor>, then I think the whole Transaction thing won't be necessay, will it?

Re: Transaction inside an IUpdater

ridaabderrahmane — Mon, 13 Jan 2020 15:16:59 GMT

Thanks Bro, the method GetVerticalProjectionPoint did solve my problem so no need to create a subtransaction and roll it back to get a point from the floor thanks a lot.

The implementation looks like this:

BoundingBoxXYZ boundingBox = floor.get_BoundingBox(null);
XYZ modelOrigin = (boundingBox.Min + boundingBox.Max) / 2.0;

XYZ floorCenter = floor.GetVerticalProjectionPoint(modelOrigin, FloorFace.Bottom);