New patch in my branch for inclusion in trunk and 3.1 branch

I made a minor change to the editor. Specifically, instead of being forced to type (and remember) the name of a prototype definition, you will now be able to select from a list of valid prototype definition names.

Let me know if you have any concerns.

TIA

Jim

Hi Jim,

Excellent. I have been meaning to do that for years!

Does it allow you to override the text and enter the name of a non-existent prototype? You may want to use the name of a prototype that you have not created yet. Or the name of a prototype that only exists in an Extension, not in the main module.

Rgds,
Brent.

*********** REPLY SEPARATOR ***********

On 27/01/2010 at 10:19 AM jimcot wrote:


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

I thought about that, but decided it wasn’t worth the effort. It’s easy enough to go declare the prototype first, even if you don’t put anything in it.

I’m going to do the same for ‘send to location’ as well as anywhere else it makes sense. Attempting to build/edit a module I am finding a lot of places where a small change improves module creation/updates immensely.

Modules already exist that add prototypes to pieces in the main module, where the prototype is only defined in an extension. It allows you to future-proof the module by allowing extensions to add functionality to counters defined in the core module, without having to provide a completely new counter set.

It shouldn’t be hard to do, just make it an Editable Combo-box.

Rgds,
B.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

That makes sense. I’ll go ahead and change it. Hold off on merging until I post here again saying it’s ready.

tia.

Jim.

Ok, you can now enter text in the combo box as well as select from a list of existing prototype definitions. Take a look and let me know if it’s ok now.

That’s looking good.

One funny I found was that if you delete a prototype definition, it still appears as an option in the drop-down list. New prototype definitions appear correctly.

B.

*********** REPLY SEPARATOR ***********

On 28/01/2010 at 12:52 PM jimcot wrote:


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

Is it the one you defined for that trait? Or is it still hanging around internally even though it’s deleted from the module’s tree?

I added three new prototype definitions A, B, C. Go to a prototype and all 3 correctly appear, select A. Close the prototype. Delete prototype definition C. Go back to a prototype and C still appears in the drop down list.

B.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

Yea, when you delete a prototype from the module, Vassal doesn’t delete it from it’s internal data store. I’ll have to fix that as well.

Ok, I fixed it to remove prototype definitions that are deleted. You shouldn’t see any more phantom names in the list.