style specific fields are now editable&saveable

git-svn-id: svn://svn.code.sf.net/p/magicseteditor/code/trunk@392 0fc631ac-6414-0410-93d0-97cfa31319b6

src/data/card.cpp

@@ -12,6 +12,7 @@
 #include <data/field.hpp>
 #include <util/error.hpp>
 #include <util/reflect.hpp>
+#include <wx/sstream.h>
 
 DECLARE_TYPEOF_COLLECTION(FieldP);
 DECLARE_TYPEOF_NO_REV(IndexMap<FieldP COMMA ValueP>);
@@ -51,6 +52,48 @@ void mark_dependency_member(const Card& card, const String& name, const Dependen
 IMPLEMENT_REFLECTION(Card) {
 	REFLECT(stylesheet);
 	REFLECT(notes);
+	REFLECT_NO_SCRIPT(extra_data);
 	REFLECT_NAMELESS(data);
 }
 
+// ----------------------------------------------------------------------------- : Styling
+
+// TODO : this is practically the same as Set::Styling, maybe somehow abstract it
+
+// Extra card data, for a specific stylesheet
+/* The data is not read immediatly, because we do not know the stylesheet */
+class Card::Styling : public IntrusivePtrBase<Card::Styling> {
+  public:
+	/// The values on the extra card fields of the card.
+	/** The indices should correspond to the extra_card_fields in the StyleSheet */
+	IndexMap<FieldP, ValueP> extra_data;
+	/// Unparsed extra_data
+	String unread_data;
+	DECLARE_REFLECTION();
+};
+
+IndexMap<FieldP, ValueP>& Card::extraDataFor(const StyleSheet& stylesheet) {
+	StylingP& styling = extra_data[stylesheet.name()];
+	if (!styling) {
+		styling = new_intrusive<Styling>();
+		styling->extra_data.init(stylesheet.extra_card_fields);
+	} else if (!styling->unread_data.empty() || (styling->extra_data.empty()) && !stylesheet.extra_card_fields.empty()) {
+		// we delayed the reading of the data, read it now
+		styling->extra_data.init(stylesheet.extra_card_fields);
+		Reader reader(new_shared1<wxStringInputStream>(styling->unread_data), _("extra card values for ") + stylesheet.stylesheetName());
+		reader.handle_greedy(styling->extra_data);
+		styling->unread_data.clear();
+	}
+	return styling->extra_data;
+}
+
+// custom reflection : read into unread_data
+template <> void Reader::handle(Card::Styling& s) {
+	handle(s.unread_data);
+}
+template <> void Writer::handle(const Card::Styling& s) {
+	handle(s.extra_data);
+}
+// for the love of god, don't depend on styling
+template <> void GetMember::handle(const Card::Styling& s) {}
+template <> void GetDefaultMember::handle(const Card::Styling& s) {}

src/data/card.hpp

@@ -34,15 +34,18 @@ class Card : public IntrusivePtrVirtualBase {
 	/// The values on the fields of the card.
 	/** The indices should correspond to the card_fields in the Game */
 	IndexMap<FieldP, ValueP> data;
-	/// The values on the extra fields of the card.
-	/** The indices should correspond to the extra_card_fields in the StyleSheet */
-	IndexMap<FieldP, ValueP> extra_data;
 	/// Notes for this card
 	String notes;
 	/// Alternative style to use for this card
 	/** Optional; if not set use the card style from the set */
 	StyleSheetP stylesheet;
 	
+	/// Extra values for specitic stylesheets, indexed by stylesheet name
+	DECLARE_POINTER_TYPE(Styling);
+	map<String, StylingP> extra_data;
+	/// Styling information for a particular stylesheet
+	IndexMap<FieldP, ValueP>& extraDataFor(const StyleSheet& stylesheet) ;
+	
 	/// Get the identification of this card, an identification is something like a name, title, etc.
 	/** May return "" */
 	String identification() const;

src/data/stylesheet.cpp

@@ -85,9 +85,9 @@ IMPLEMENT_REFLECTION(StyleSheet) {
 	REFLECT(extra_card_fields);
 	REFLECT_IF_READING {
 		if (extra_card_style.init(extra_card_fields)) {
-			// make sure the extra_card_fields are not editable and savable
+			// if a value is not editable, don't save it
 			FOR_EACH(f, extra_card_fields) {
-				f->editable = f->save_value = false;
+				if (!f->editable) f->save_value = false;
 			}
 		}
 	}