Community
It is good you showed your code in details. It looks like your code needs big refactore, IMO.
The major issue is that you pass DBObject/Entity around between/outside transactions, which is big NO in AutoCAD .NET API coding. Entity taken outside transaction context may not behave as it should. For example, a blockreference to a dynamic block definition would only have its IsDynamic property being true when the BlockReference stays inside a Transaction. The situation you described is likely because you pass entities around outside transaction instead of ObjectIDs.
The rule of thumb is alway pass ObjectId around and only work with Entity within a transaction. You can use Entity without Transaction if you do not intend to add it into database and you are responsible to dispose it. BTW, you code in the Jig creates Line entity in order to calculate angle without dispose it. That means, a huge number of lines would be created in momory if the user keep dragging the mouse, which triggers Sampler().
Why not just run the Jig on the opened for read entities, e.g. (not checked or compiled - just off the top of my head here - also please read my Fenton comments)
PromptPointOptions ppo = new PromptPointOptions("\nSelect base point of the rotation");
PromptPointResult ppr = ed.GetPoint(ppo);
if (ppr.Status != PromptStatus.OK)
return;
Point3d center = ppr.Value;
using (Transaction trans = db.TransactionManager.StartOpenCloseTransaction())
{
foreach (SelectedObject so in psr.Value)
toRotate.Add(trans.GetObject(so.ObjectId, OpenMode.ForRead) as Entity);
JigReferenceRotator jig = new JigReferenceRotator(center);
ed.Drag(jig);
jig.Axis = false;
PromptResult pr;
List<Entity> reference;
while (true)
{
reference = toRotate;
// Fenton: why clone??
// toRotate = EntityListClone(toRotate);
jig.toRotate = toRotate;
pr = ed.Drag(jig);
if (pr.Status != PromptStatus.OK)
{
// Fenton: there is only one reason to lock, that is if you are edting an entity from the application context. Also, jigs don't work at all well from the application context so this is pointless
// using (DocumentLock dl = doc.LockDocument())
{
// fenton: not needed
// using (Transaction trans = db.TransactionManager.StartTransaction())
{
// Fenton: because we are rotating the originals no need to delete
//foreach (Entity ent in toRotate)
//trans.GetObject(ent.ObjectId, OpenMode.ForWrite).Erase();
trans.Commit();
}
}
// Fenton: if we need to call regen, we are doing something wrong somewhere
//ed.Regen();
break;
}
// Fenton: if we need to call regen, we are doing something wrong somewhere ed.Regen();
}
}
// Fenton: jig code updates this function should be replaced from this:
protected override bool WorldDraw(Autodesk.AutoCAD.GraphicsInterface.WorldDraw draw)
{
if (Axis)
draw.Geometry.Draw(axis);
else
{
draw.Geometry.Draw(axis);
double angle = firstAngle - new Line(axis.BasePoint, axis.SecondPoint).Angle;
using (Transaction trans = Application.DocumentManager.MdiActiveDocument.Database.
TransactionManager.StartTransaction())
{
foreach (Entity ent in toRotate)
{
(trans.GetObject(ent.ObjectId, OpenMode.ForWrite) as Entity).TransformBy(
Matrix3d.Rotation(oldAngle - angle, Vector3d.ZAxis, center));
draw.Geometry.Draw(ent);
}
trans.Commit();
}
oldAngle = angle;
}
return true;
}
}
// Fenton: to this (BTW ****NEVER****EVER*** use transactions in a Jig)
protected override bool WorldDraw(Autodesk.AutoCAD.GraphicsInterface.WorldDraw draw)
{
if (Axis)
draw.Geometry.Draw(axis);
else
{
draw.Geometry.Draw(axis);
double angle = firstAngle - new Line(axis.BasePoint, axis.SecondPoint).Angle;
foreach (Entity ent in toRotate)
{
(trans.GetObject(ent.ObjectId, OpenMode.ForWrite) as Entity).TransformBy(
Matrix3d.Rotation(oldAngle - angle, Vector3d.ZAxis, center));
draw.Geometry.Draw(ent);
}
}
oldAngle = angle;
}
return true;
}
}
Fenton Webb
AutoCAD Engineering
Autodesk
