Card data in images (start)

* Add uuid implementation

* Simplify uid implementation

* Check for uid conflicts upon adding/copying cards

* Remove unnecessary checks

these are already checked as part of the first loop, since they were added to set.cards

* Implement card linking

* refactor to avoid triple loop

* Start link visualization

* Continue link visualization

* formatting

* add missing locale entries

* improve layout

* improve UI refresh

* copy links when copying cards

* implement second face editor

* refactor using macros

* get references

* pasting multiple cards and links

* start refactoring editors

* continue refactoring editors

* continue refactoring editors again

* finish refactoring editors

* refresh card list on link editor event

* tweak event handling

* Add insert_image script function

* change parameter type

* add script functions to get cards

* add dimensions_of script function

calculate both dimensions simultaneously while rendering the image only once.

* save value even if not editable

* add get_mse_locale script function

* Change zoom scale choices

* bug fixes

* Scale symbol spacing with font size

* sync locales

* standardize line ending

* handle links in new_card

* initial

waiting on wxWidgets 3.3.1

* update comment

* store rect in image filename

* store rect in temp file name

* Update package.cpp

* change line endings

src/data/field.cpp

@@ -113,9 +113,14 @@ Style::Style(const FieldP& field)
   , visible(true)
   , automatic_side(AUTO_UNKNOWN)
   , content_dependent(false)
-{}
+{
+  field->styleP = this;
+}
 
-Style::~Style() {}
+Style::~Style()
+{
+  fieldP->styleP = nullptr;
+}
 
 IMPLEMENT_REFLECTION(Style) {
   REFLECT(z_index);
@@ -133,6 +138,7 @@ IMPLEMENT_REFLECTION(Style) {
 
 void init_object(const FieldP& field, StyleP& style) {
   if (!style) style = field->newStyle();
+  field->styleP = style;
 }
 template <> StyleP read_new<Style>(Reader&) {
   throw InternalError(_("IndexMap contains nullptr StyleP the application should have crashed already"));
@@ -262,6 +268,18 @@ void mark_dependency_member(const Style& style, const String& name, const Depend
   style.markDependencyMember(name,dep);
 }
 
+String Style::getRect() {
+  return _("---") +
+         wxString::Format(wxT("%i"), (int)(left)) +
+         _("-") +
+         wxString::Format(wxT("%i"), (int)(top)) +
+         _("-") +
+         wxString::Format(wxT("%i"), (int)(width)) +
+         _("-") +
+         wxString::Format(wxT("%i"), (int)(height)) +
+         _("---");
+}
+
 // ----------------------------------------------------------------------------- : StyleListener
 
 void Style::addListener(StyleListener* listener) {

src/data/field.hpp

@@ -63,6 +63,7 @@ public:
   OptionalScript  import_script;             ///< The script to apply to the supplied value, when creating a new card.
   Dependencies    dependent_scripts;         ///< Scripts that depend on values of this field
   String          package_relative_filename;
+  StyleP          styleP;                    ///< Style for this field, should have the right type! Can be null.
 
   /// Creates a new Value corresponding to this Field
   virtual ValueP newValue() = 0;
@@ -163,6 +164,9 @@ public:
   /** change_info is a subset of StyleChange flags */
   void tellListeners(int changes);
   
+  /// Store where on the card the field goes, to save it in filenames
+  String getRect();
+
 private:
   DECLARE_REFLECTION_VIRTUAL();
   /// Things that are listening to changes in this style

src/data/format/clipboard.cpp

@@ -71,6 +71,7 @@ CardsDataObject::CardsDataObject(const SetP& set, const vector<CardP>& cards) {
   }
   WrappedCards data = { set->game.get(), set->game->name(), cards };
   SetText(serialize_for_clipboard(*set, data));
+  queue_message(MESSAGE_WARNING, GetText());
   // restore cards
   for (size_t i = 0 ; i < cards.size() ; ++i) {
     if (has_styling[i]) {
@@ -140,12 +141,12 @@ KeywordP KeywordDataObject::getKeyword(const SetP& set) {
 // ----------------------------------------------------------------------------- : Card on clipboard
 
 CardsOnClipboard::CardsOnClipboard(const SetP& set, const vector<CardP>& cards) {
-  // Conversion to text format
+  // Conversion to image format
-    // TODO
-    //Add( new TextDataObject(_("card"))) 
-  // Conversion to bitmap format
     if (cards.size() == 1) {
-      Add(new wxBitmapDataObject(export_bitmap(set, cards[0])));
+      Add(new wxImageDataObject(export_image(set, cards[0])));
+    }
+    else if (cards.size() < 6) {
+      Add(new wxImageDataObject(export_image(set, cards, true, 0, 1.0, 0.0)));
     }
     else if (cards.size() < 6) {
       Add(new wxBitmapDataObject(export_bitmap(set, cards, true, 0, 1.0, 0.0)));

src/data/format/formats.hpp

@@ -88,20 +88,18 @@ FileFormatP mtg_editor_file_format();
 
 // ----------------------------------------------------------------------------- : Other ways to export
 
-/// Export images for each card in a set to a list of files
+/// Generate a wxBitmap of one or more cards
-void export_images(Window* parent, const SetP& set);
-
-/// Export the image for each card in a list of cards
-void export_images(const SetP& set, const vector<CardP>& cards,
-                   const String& path, const String& filename_template, FilenameConflicts conflicts);
-
-/// Export the image of a single card
-void export_image(const SetP& set, const CardP& card, const String& filename);
-
-/// Generate a bitmap image of a card
 Bitmap export_bitmap(const SetP& set, const CardP& card);
-Bitmap export_bitmap(const SetP& set, const CardP& card, const double zoom, const Radians angle_radians);
+Bitmap export_bitmap(const SetP& set, const CardP& card, const double zoom, const Radians angle_radians = 0.0);
-Bitmap export_bitmap(const SetP& set, const vector<CardP>& cards, bool scale_to_lowest_dpi, int padding, const double zoom, const Radians angle_radians);
+Bitmap export_bitmap(const SetP& set, const vector<CardP>& cards, bool scale_to_lowest_dpi, int padding, const double zoom, const Radians angle_radians = 0.0);
+
+/// Generate a wxImage of one or more cards
+Image export_image(const SetP& set, const CardP& card);
+Image export_image(const SetP& set, const CardP& card, const double zoom, const Radians angle_radians = 0.0);
+Image export_image(const SetP& set, const vector<CardP>& cards, bool scale_to_lowest_dpi, int padding, const double zoom, const Radians angle_radians = 0.0);
+
+/// Export the image of a single card to a given filename
+void export_image(const SetP& set, const CardP& card, const String& filename);
 
 /// Export a set to Magic Workstation format
 void export_mws(Window* parent, const SetP& set);

src/data/format/image.cpp

@@ -9,6 +9,8 @@
 #include <util/prec.hpp>
 #include <util/tagged_string.hpp>
 #include <data/format/formats.hpp>
+#include <data/format/clipboard.hpp>
+#include <data/game.hpp>
 #include <data/set.hpp>
 #include <data/card.hpp>
 #include <data/stylesheet.hpp>
@@ -17,18 +19,14 @@
 #include <render/card/viewer.hpp>
 #include <wx/filename.h>
 
-// ----------------------------------------------------------------------------- : Single card export
+#define wxIMAGE_OPTION_PNG_DESCRIPTION wxString("PngDescription")
 
-void export_image(const SetP& set, const CardP& card, const String& filename) {
+// ----------------------------------------------------------------------------- : Card export
-  Image img = export_bitmap(set, card).ConvertToImage();
-  img.SaveFile(filename);  // can't use Bitmap::saveFile, it wants to know the file type
-              // but image.saveFile determines it automagicly
-}
 
 class UnzoomedDataViewer : public DataViewer {
 public:
   UnzoomedDataViewer();
-  UnzoomedDataViewer(double zoom, double angle);
+  UnzoomedDataViewer(double zoom, Radians angle);
   virtual ~UnzoomedDataViewer() {};
   Rotation getRotation() const override;
 private:
@@ -65,6 +63,8 @@ Rotation UnzoomedDataViewer::getRotation() const {
   }
 }
 
+// ----------------------------------------------------------------------------- : wxBitmap export
+
 Bitmap export_bitmap(const SetP& set, const CardP& card) {
   if (!set) throw Error(_("no set"));
   UnzoomedDataViewer viewer = UnzoomedDataViewer();
@@ -83,9 +83,9 @@ Bitmap export_bitmap(const SetP& set, const CardP& card) {
   return bitmap;
 }
 
-Bitmap export_bitmap(const SetP& set, const CardP& card, const double zoom, const Radians angle = 0.0) {
+Bitmap export_bitmap(const SetP& set, const CardP& card, const double zoom, const Radians angle_radians) {
   if (!set) throw Error(_("no set"));
-  UnzoomedDataViewer viewer = UnzoomedDataViewer(zoom, angle);
+  UnzoomedDataViewer viewer = UnzoomedDataViewer(zoom, angle_radians);
   viewer.setSet(set);
   viewer.setCard(card);
   // size of cards
@@ -102,7 +102,7 @@ Bitmap export_bitmap(const SetP& set, const CardP& card, const double zoom, cons
 }
 
 // put multiple card images into one bitmap
-Bitmap export_bitmap(const SetP& set, const vector<CardP>& cards, bool scale_to_lowest_dpi, int padding, const double zoom, const Radians angle = 0.0) {
+Bitmap export_bitmap(const SetP& set, const vector<CardP>& cards, bool scale_to_lowest_dpi, int padding, const double zoom, const Radians angle_radians) {
   if (!set) throw Error(_("no set"));
   vector<Bitmap> bitmaps;
   int width = 0;
@@ -121,7 +121,7 @@ Bitmap export_bitmap(const SetP& set, const vector<CardP>& cards, bool scale_to_
       double dpi = max(set->stylesheetFor(card).card_dpi, 150.0);
       scaled_zoom *= lowest_dpi / dpi;
     }
-    UnzoomedDataViewer viewer = UnzoomedDataViewer(scaled_zoom, angle);
+    UnzoomedDataViewer viewer = UnzoomedDataViewer(scaled_zoom, angle_radians);
     viewer.setSet(set);
     viewer.setCard(card);
     RealSize size = viewer.getRotation().getExternalSize();
@@ -152,8 +152,39 @@ Bitmap export_bitmap(const SetP& set, const vector<CardP>& cards, bool scale_to_
   return global_bitmap;
 }
 
-// ----------------------------------------------------------------------------- : Multiple card export
+// ----------------------------------------------------------------------------- : wxImage export
 
+Image export_image(const SetP& set, const CardP& card) {
+  Bitmap bitmap = export_bitmap(set, card);
+  Image img = bitmap.ConvertToImage();
+  vector<CardP> cards = { card };
+  CardsDataObject data(set, cards);
+  img.SetOption(wxIMAGE_OPTION_PNG_DESCRIPTION, _("<mse-data-start>") + data.GetText() + _("<mse-data-end>"));
+  return img;
+}
+
+Image export_image(const SetP& set, const CardP& card, const double zoom, const Radians angle_radians) {
+  Bitmap bitmap = export_bitmap(set, card, zoom, angle_radians);
+  Image img = bitmap.ConvertToImage();
+  vector<CardP> cards = { card };
+  CardsDataObject data(set, cards);
+  img.SetOption(wxIMAGE_OPTION_PNG_DESCRIPTION, _("<mse-data-start>") + data.GetText() + _("<mse-data-end>"));
+  return img;
+}
+
+Image export_image(const SetP& set, const vector<CardP>& cards, bool scale_to_lowest_dpi, int padding, const double zoom, const Radians angle_radians) {
+  Bitmap bitmap = export_bitmap(set, cards, scale_to_lowest_dpi, padding, zoom, angle_radians);
+  Image img = bitmap.ConvertToImage();
+  CardsDataObject data(set, cards);
+  img.SetOption(wxIMAGE_OPTION_PNG_DESCRIPTION, _("<mse-data-start>") + data.GetText() + _("<mse-data-end>"));
+  return img;
+}
+
+void export_image(const SetP& set, const CardP& card, const String& filename) {
+  Image img = export_image(set, card);
+  img.SaveFile(filename); // can't use Bitmap::saveFile, it wants to know the file type
+                          // but image.saveFile determines it automagicly
+}
 
 void export_images(const SetP& set, const vector<CardP>& cards,
                    const String& path, const String& filename_template, FilenameConflicts conflicts)

src/gui/add_json_window.cpp

@@ -18,6 +18,7 @@
 #include <data/card.hpp>
 #include <data/stylesheet.hpp>
 #include <data/action/set.hpp>
+#include <data/format/formats.hpp>
 #include <gui/add_json_window.hpp>
 #include <script/functions/json.hpp>
 #include <script/functions/construction_helper.hpp>
@@ -171,6 +172,19 @@ void AddJSONWindow::onBrowseFiles(wxCommandEvent&) {
 }
 
 void AddJSONWindow::onOk(wxCommandEvent&) {
+
+
+
+  // debug test shit
+  export_image(set, set->cards.front(), "C:\\Users\\Oli\\Desktop\\tetest\\test.png");
+  auto extImg = make_intrusive<ExternalImage>("C:/Users/Oli/Desktop/tetest/test.png");
+  Image img = extImg->generate(GeneratedImage::Options(0, 0, set->stylesheet.get(), set.get()));
+  if (img.HasOption(wxIMAGE_OPTION_FILENAME))  queue_message(MESSAGE_ERROR, img.GetOption(wxIMAGE_OPTION_FILENAME));
+  else queue_message(MESSAGE_ERROR, _("no dice"));
+  return;
+
+
+
   /// Perform the import
   wxBusyCursor wait;
   // Read the file

src/gui/value/image.cpp

@@ -58,7 +58,10 @@ void ImageValueEditor::sliceImage(const Image& image, const String& filename, co
   // clicked ok?
   if (s.ShowModal() == wxID_OK) {
     // store the image into the set
-    LocalFileName new_image_file = getLocalPackage().newFileName(field().name, settings.internal_image_extension ? _(".png") : _("")); // a new unique name in the package
+    StyleP style = field().styleP;
+    String rect = style ? style->getRect() : _("");
+    String extension = settings.internal_image_extension ? _(".png") : _("");
+    LocalFileName new_image_file = getLocalPackage().newFileName(field().name, rect + extension); // a new unique name in the package
 
     // Specify a desired size based on the stylesheet and a scale multiplier defined within the user's settings.
     // Storing at a greater than 100% resolution allows for better exports >100%, but may change how images look when filters (sharpen) are applied.

src/script/functions/export.cpp

@@ -435,7 +435,7 @@ SCRIPT_FUNCTION(write_image_file) {
   Image image;
   GeneratedImage::Options options(width, height, ei.export_template.get(), ei.set.get());
   if (card) {
-    image = conform_image(export_bitmap(ei.set, card->getValue()).ConvertToImage(), options);
+    image = conform_image(export_image(ei.set, card->getValue()), options);
   } else {
     image = input->toImage()->generateConform(options);
   }

src/script/functions/image.cpp

@@ -38,10 +38,10 @@ SCRIPT_FUNCTION(to_card_image) {
   SCRIPT_PARAM_DEFAULT(bool, use_user_settings, false);
   if (use_user_settings) {
     // Use the User's Preferences for Export Zoom and Angle settings.
-    return make_intrusive<ArbitraryImage>(export_bitmap(set, input).ConvertToImage());
+    return make_intrusive<ArbitraryImage>(export_image(set, input));
   } else {
     // Use the provided (or defaulted) Zoom and Angle.
-    return make_intrusive<ArbitraryImage>(export_bitmap(set, input, (zoom / 100), deg_to_rad(angle)).ConvertToImage());
+    return make_intrusive<ArbitraryImage>(export_image(set, input, (zoom / 100), deg_to_rad(angle)));
   }
 }
 

src/util/io/package.cpp

@@ -272,6 +272,11 @@ String Package::nameOut(const String& file) {
   } else {
     // create temp file
     String name = wxFileName::CreateTempFileName(_("mse"));
+    String rect = LocalFileName::getRect(file);
+    if (!rect.empty()) {
+      if (name.Contains(".")) name = name.BeforeLast('.') + rect + _(".") + name.AfterLast('.');
+      else name = name + rect;
+    }
     it->second.tempName = name;
     return name;
   }
@@ -361,7 +366,7 @@ LocalFileName LocalFileName::fromReadString(String const& fn, String const& pref
   if (!fn.empty() && clipboard_package()) {
     // copy file into current package
     try {
-      LocalFileName local_name = clipboard_package()->newFileName(_("image"),_("")); // a new unique name in the package, assume it's an image
+      LocalFileName local_name = clipboard_package()->newFileName(_("image"), getRect(fn)); // a new unique name in the package, assume it's an image
       auto out_stream = clipboard_package()->openOut(local_name);
       auto in_stream  = Package::openAbsoluteFile(fn);
       out_stream->Write(*in_stream); // copy

src/util/io/package.hpp

@@ -53,6 +53,15 @@ public:
 
   inline String const& toStringForKey() const { return fn; }
 
+  /// Retreive a rect from a filename
+  static String getRect(const String& file) {
+      size_t first = file.find(_("---"));
+      if (first == String::npos) return _("");
+      size_t last = file.find(_("---"), first+3);
+      if (first == last) return _("");
+      return file.substr(first, last + 3 - first);
+  }
+
 private:
   LocalFileName(const wxString& fn) : fn(fn) {}
   String fn;