bsimser / Inventory-Pro

https://devdog.io
MIT License
2 stars 0 forks source link

Make a wrapper for properties (same as currencies) #111

Closed bsimser closed 8 years ago

bsimser commented 9 years ago

Original report by Joris Huijbregts (Bitbucket: jjahuijbregts, ).


Currencies are currently wrapped to maintain 1 reference object of each type, making changes very easy.

Current properties are copied onto the object, when a name changes the property might not change along with it, as it has to be re-copied. + harder to replicate in other area's in code.

Create wrapper:

public class InventoryPropertyLookup { protected uint _propertyID; public InventoryProperty{get { // get it } }

public float amount;
/// All the amount based stuff here -> Separate it from the defition (InventoryItemProperty)

}

bsimser commented 8 years ago

Original comment by Joris Huijbregts (Bitbucket: jjahuijbregts, ).


Properties wrapped with lookup fixes #111 + updated setup wizard to detect faulty properties

bsimser commented 8 years ago

Original comment by Joris Huijbregts (Bitbucket: jjahuijbregts, ).


Properties wrapped with lookup fixes #111 + updated setup wizard to detect faulty properties