Community
BlockTableRecord tBlock = (BlockTableRecord) bTable["RoomID"].GetObject(OpenMode.ForWrite);
BlockReference newRef = new BlockReference(refPoint, tBlock.ObjectId);
tBlock.AppendEntity(newRef);
trans1.AddNewlyCreatedDBObject(newRef, true);
The above lines of code (ignoring the other property stuff in between) insert the RoomID block into the RoomID block.
Open a Write reference to the block ModelSpace, and do ModelSpace.AppendEntity(newref) instead of tblock.append.
Dave O. 
Well, I'm assuming you have a trans1.commit below there somewhere, right? Otherwise it looks like it should work. Have you stepped through it to see that there are attributes in the collection?
I am attaching an example I clipped out of my code (and ran through a c# converter, since I use vb). It is a Sub which sets the Textstring of a single attribute when passed the ObjectID of the BlockReference, the Tag of the AttributeReference to set, and the string to set to the TextString property. You should be able to use it to see where you went wrong, or modify it to suit your purpose, otherwise you'll need to post the whole method in question so I (or anyone else here) can try and track down the problem.
Edited by: chiefbraincloud on Apr 29, 2009 1:36 PM
I just noticed the conversion to c# added a comment that says it changed my 'Exit For' to 'Break', so I think you need to fix that.
Dave O.
Transaction trans1 = currentDoc.TransactionManager.StartTransaction();
I think you should be using the transaction manager that comes from the Database object. That may be the problem.
GetValueFromDatabase(ref attRef); // This method assigns attRef.TextString from database
Correctly passing an Object that you already have open for write can be tricky. I avoid it like the plague. I always pass ObjectID and do the transaction and Open ForWrite in the subroutine. Perhaps you could change GetValueFromDatabase to a function that returns the proper textstring value, then set the textstring value here in this method, since you already have the reference open, or alternatively I would not open the reference in this method, and instead pass the objectID to GetValueFromDatabase and open it there.
Try changing the TransactionManager reference to the one from Database... If that doesn't work then the problem is probably in the GetVauleFromDatabase sub.
I think you should be using the transaction manager that comes from the Database object. That may be the problem.
GetValueFromDatabase(ref attRef); // This method assigns attRef.TextString from database
Correctly passing an Object that you already have open for write can be tricky. I avoid it like the plague. I always pass ObjectID and do the transaction and Open ForWrite in the subroutine. Perhaps you could change GetValueFromDatabase to a function that returns the proper textstring value, then set the textstring value here in this method, since you already have the reference open, or alternatively I would not open the reference in this method, and instead pass the objectID to GetValueFromDatabase and open it there.
Try changing the TransactionManager reference to the one from Database... If that doesn't work then the problem is probably in the GetVauleFromDatabase sub.
Dave O.
That sounds like it may be a document lock issue, which is sort of wierd in itself since it works on the created event, but try adding a document lock to the beginning and make sure you dispose it afterwards.
Dave O.
