Fixed Actions for TextCtrl, actions used to apply to the wrong value.

Changed some TABs to spaces in macros, that should end the conflicts because we use different tab sizes (4 vs 8)

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

src/data/action/value.cpp

@@ -35,6 +35,7 @@ class SimpleValueAction : public ValueAction {
 	
 	virtual void perform(bool to_undo) {
 		swap(static_cast<T&>(*valueP).*member, new_value);
+		valueP->onAction(*this, to_undo); // notify value
 	}
 	
 	virtual bool merge(const Action& action) {
@@ -54,9 +55,9 @@ class SimpleValueAction : public ValueAction {
 	typename T::ValueType T::*member;
 };
 
-ValueAction* value_action(const ChoiceValueP&         value, const Defaultable<String>& new_value) { return new SimpleValueAction<ChoiceValue, true> (value, new_value, &ChoiceValue::value); }
+ValueAction* value_action(const ChoiceValueP&         value, const Defaultable<String>& new_value) { return new SimpleValueAction<ChoiceValue,         true> (value, new_value, &ChoiceValue::value); }
 ValueAction* value_action(const MultipleChoiceValueP& value, const Defaultable<String>& new_value) { return new SimpleValueAction<MultipleChoiceValue, false>(value, new_value, &MultipleChoiceValue::value); }
-ValueAction* value_action(const ColorValueP&          value, const Defaultable<Color>&  new_value) { return new SimpleValueAction<ColorValue, true> (value, new_value, &ColorValue::value); }
+ValueAction* value_action(const ColorValueP&          value, const Defaultable<Color>&  new_value) { return new SimpleValueAction<ColorValue,          true> (value, new_value, &ColorValue::value); }
 ValueAction* value_action(const ImageValueP&          value, const FileName&            new_value) { return new SimpleValueAction<ImageValue,          false>(value, new_value, &ImageValue::filename); }
 ValueAction* value_action(const SymbolValueP&         value, const FileName&            new_value) { return new SimpleValueAction<SymbolValue,         false>(value, new_value, &SymbolValue::filename); }
 
@@ -76,6 +77,7 @@ void TextValueAction::perform(bool to_undo) {
 	swap(value().value, new_value);
 	value().last_update.update();
 	swap(selection_end, new_selection_end);
+	valueP->onAction(*this, to_undo); // notify value
 }
 
 bool TextValueAction::merge(const Action& action) {

src/data/field.cpp

@@ -127,10 +127,13 @@ Value::~Value() {}
 IMPLEMENT_REFLECTION_NAMELESS(Value) {
 }
 
+bool Value::equals(const Value* that) {
+	return this == that;
+}
+
 void init_object(const FieldP& field, ValueP& value) {
 	if (!value) value = field->newValue(field);
 }
 template <> ValueP read_new<Value>(Reader&) {
 	throw InternalError(_("IndexMap contains nullptr ValueP the application should have crashed already"));
 }
-

src/data/field.hpp

@@ -21,6 +21,7 @@ DECLARE_POINTER_TYPE(Style);
 DECLARE_POINTER_TYPE(Value);
 class Context;
 class Dependency;
+class Action;
 
 // for DataViewer/editor
 class DataViewer; class DataEditor;
@@ -134,6 +135,14 @@ class Value {
 	virtual String toString() const = 0;
 	/// Apply scripts to this value, return true if the value has changed
 	virtual bool update(Context&) { last_script_update.update(); return false; }
+	/// This value has been updated by an action
+	/** Does nothing for most Values, only FakeValues can update underlying data */
+	virtual void onAction(Action& a, bool undone) {}
+	/// Is this value the same as some other value (for the same field&card)
+	/** Has behaviour other than == for FakeTextValue.
+	 *  In that case, afterwards this becomes equal to that if they use the same underlying object.
+	 */
+	virtual bool equals(const Value* that);
 	
   private:
 	DECLARE_REFLECTION_VIRTUAL();

src/data/field/text.cpp

@@ -105,3 +105,18 @@ bool TextValue::update(Context& ctx) {
 IMPLEMENT_REFLECTION_NAMELESS(TextValue) {
 	if (fieldP->save_value || tag.scripting()) REFLECT_NAMELESS(value);
 }
+
+// ----------------------------------------------------------------------------- : FakeTextValue
+
+void FakeTextValue::onAction(Action& a, bool undone) {
+	*underlying = value;
+}
+
+bool FakeTextValue::equals(const Value* that) {
+	if (this == that) return true;
+	const FakeTextValue* thatT = dynamic_cast<const FakeTextValue*>(that);
+	if (!thatT || underlying != thatT->underlying) return false;
+	// update the value
+	value = *underlying;
+	return true;
+}

src/data/field/text.hpp

@@ -98,5 +98,22 @@ class TextValue : public Value {
 	DECLARE_REFLECTION();
 };
 
+// ----------------------------------------------------------------------------- : TextValue
+
+/// A 'fake' TextValue that is used to edit some other string
+/** Used by TextCtrl */
+class FakeTextValue : public TextValue {
+  public:
+	inline FakeTextValue(const TextFieldP& field, String* underlying)
+		: TextValue(field), underlying(underlying) {}
+	
+	String* const underlying; ///< The underlying actual value
+	
+	/// Update underlying data
+	virtual void onAction(Action& a, bool undone);
+	/// Editing the same underlying value?
+	virtual bool equals(const Value* that);
+};
+
 // ----------------------------------------------------------------------------- : EOF
 #endif

src/data/game.hpp

@@ -34,21 +34,21 @@ class Game : public Packaged {
   public:
 	Game();
 	
-	OptionalScript          init_script;			///< Script of variables available to other scripts in this game
-	vector<FieldP>          set_fields;				///< Fields for set information
-	IndexMap<FieldP,StyleP> default_set_style;		///< Default style for the set fields, because it is often the same
-	vector<FieldP>          card_fields;			///< Fields on each card
-	vector<StatsDimensionP> statistics_dimensions;	///< (Additional) statistics dimensions
-	vector<StatsCategoryP>  statistics_categories;	///< (Additional) statistics categories
-	
+	OptionalScript          init_script;            ///< Script of variables available to other scripts in this game
+	vector<FieldP>          set_fields;             ///< Fields for set information
+	IndexMap<FieldP,StyleP> default_set_style;      ///< Default style for the set fields, because it is often the same
+	vector<FieldP>          card_fields;            ///< Fields on each card
+	vector<StatsDimensionP> statistics_dimensions;  ///< (Additional) statistics dimensions
+	vector<StatsCategoryP>  statistics_categories;  ///< (Additional) statistics categories
+
 	bool                    has_keywords;           ///< Does this game use keywords?
 	vector<KeywordParamP>   keyword_parameter_types;///< Types of keyword parameters
 	vector<KeywordModeP>    keyword_modes;          ///< Modes of keywords
-	vector<KeywordP>        keywords;				///< Keywords for use in text
-	
-	Dependencies dependent_scripts_cards;			///< scripts that depend on the card list
-	Dependencies dependent_scripts_keywords;		///< scripts that depend on the keywords
-	bool dependencies_initialized;					///< are the script dependencies comming from this game all initialized?
+	vector<KeywordP>        keywords;               ///< Keywords for use in text
+
+	Dependencies dependent_scripts_cards;           ///< scripts that depend on the card list
+	Dependencies dependent_scripts_keywords;        ///< scripts that depend on the keywords
+	bool dependencies_initialized;                  ///< are the script dependencies comming from this game all initialized?
 	
 	/// Loads the game with a particular name, for example "magic"
 	static GameP byName(const String& name);

src/data/keyword.hpp

@@ -59,6 +59,7 @@ class Keyword {
 	 *  captures 2,4,... capture the parameters
 	 */
 	wxRegEx               matchRe;
+	bool                  fixed;		///< Is this keyword uneditable? (true for game keywods, false for set keywords)
 	
 	/// Prepare the expansion: (re)generate matchRe and the list of parameters.
 	/** Throws when there is an error in the input

src/data/symbol_font.cpp

@@ -119,7 +119,7 @@ Bitmap SymbolInFont::getBitmap(Context& ctx, Package& pkg, double size) {
 		actual_size = wxSize(img.GetWidth(), img.GetHeight());
 		// scale to match expected size
 		Image resampled_image((int) (actual_size.GetWidth()  * size / img_size),
-			                 (int) (actual_size.GetHeight() * size / img_size), false);
+		                      (int) (actual_size.GetHeight() * size / img_size), false);
 		if (!resampled_image.Ok()) return Bitmap(1,1);
 		resample(img, resampled_image);
 		// convert to bitmap, store for later use