diff --git a/src/layer-manager.cpp b/src/layer-manager.cpp
index 34a909a796aef25edb2f10746a98f5b29b5133eb..3e49edac5723299ab5699850b14ab31fe6c478c1 100644 (file)
--- a/src/layer-manager.cpp
+++ b/src/layer-manager.cpp
#include "desktop.h"
#include "desktop-handles.h"
#include "layer-manager.h"
-#include "prefs-utils.h"
+#include "preferences.h"
#include "ui/view/view.h"
#include "selection.h"
#include "sp-object.h"
+#include "sp-item-group.h"
#include "xml/node.h"
#include "xml/node-observer.h"
+#include "util/format.h"
+// #include "debug/event-tracker.h"
+// #include "debug/simple-event.h"
namespace Inkscape {
GQuark _labelAttr;
};
+/*
+namespace {
+
+Util::ptr_shared<char> stringify_node(Node const &node);
+
+Util::ptr_shared<char> stringify_obj(SPObject const &obj) {
+ gchar *string;
+
+ if (obj.id) {
+ string = g_strdup_printf("SPObject(%p)=%s repr(%p)", &obj, obj.id, obj.repr);
+ } else {
+ string = g_strdup_printf("SPObject(%p) repr(%p)", &obj, obj.repr);
+ }
+
+ Util::ptr_shared<char> result=Util::share_string(string);
+ g_free(string);
+ return result;
+
+}
+
+typedef Debug::SimpleEvent<Debug::Event::OTHER> DebugLayer;
+
+class DebugLayerNote : public DebugLayer {
+public:
+ DebugLayerNote(Util::ptr_shared<char> descr)
+ : DebugLayer(Util::share_static_string("layer-note"))
+ {
+ _addProperty("descr", descr);
+ }
+};
+
+class DebugLayerRebuild : public DebugLayer {
+public:
+ DebugLayerRebuild()
+ : DebugLayer(Util::share_static_string("rebuild-layers"))
+ {
+ }
+};
+
+class DebugLayerObj : public DebugLayer {
+public:
+ DebugLayerObj(SPObject const& obj, Util::ptr_shared<char> name)
+ : DebugLayer(name)
+ {
+ _addProperty("layer", stringify_obj(obj));
+ }
+};
+
+class DebugAddLayer : public DebugLayerObj {
+public:
+ DebugAddLayer(SPObject const &obj)
+ : DebugLayerObj(obj, Util::share_static_string("add-layer"))
+ {
+ }
+};
+
+
+} // end of namespace
+*/
LayerManager::LayerManager(SPDesktop *desktop)
: _desktop(desktop), _document(NULL)
_setDocument(desktop->doc());
}
+LayerManager::~LayerManager()
+{
+ _layer_connection.disconnect();
+ _document_connection.disconnect();
+ _resource_connection.disconnect();
+}
void LayerManager::setCurrentLayer( SPObject* obj )
{
if ( _desktop->currentRoot() ) {
_desktop->setCurrentLayer( obj );
- if ( prefs_get_int_attribute_limited("options.selection", "layerdeselect", 1, 0, 1) ) {
+ Inkscape::Preferences *prefs = Inkscape::Preferences::get();
+ if (prefs->getBool("/options/selection/layerdeselect", true)) {
sp_desktop_selection( _desktop )->clear();
}
}
}
void LayerManager::_rebuild() {
+// Debug::EventTracker<DebugLayerRebuild> tracker1();
+
while ( !_watchers.empty() ) {
LayerWatcher* one = _watchers.back();
_watchers.pop_back();
_clear();
- GSList const *layers=sp_document_get_resource_list(_document, "layer");
+ if (!_document) // http://sourceforge.net/mailarchive/forum.php?thread_name=5747bce9a7ed077c1b4fc9f0f4f8a5e0%40localhost&forum_name=inkscape-devel
+ return;
+
+ GSList const *layers = sp_document_get_resource_list(_document, "layer");
SPObject *root=_desktop->currentRoot();
if ( root ) {
_addOne(root);
- for ( GSList const *iter=layers ; iter ; iter = iter->next ) {
- SPObject *layer=static_cast<SPObject *>(iter->data);
+ std::set<SPGroup*> layersToAdd;
+
+ for ( GSList const *iter = layers; iter; iter = iter->next ) {
+ SPObject *layer = static_cast<SPObject *>(iter->data);
+// Debug::EventTracker<DebugLayerNote> tracker(Util::format("Examining %s", layer->label()));
+ bool needsAdd = false;
+ std::set<SPGroup*> additional;
+
+ if ( root->isAncestorOf(layer) ) {
+ needsAdd = true;
+ for ( SPObject* curr = layer; curr && (curr != root) && needsAdd; curr = SP_OBJECT_PARENT(curr) ) {
+ if ( SP_IS_GROUP(curr) ) {
+ SPGroup* group = SP_GROUP(curr);
+ if ( group->layerMode() == SPGroup::LAYER ) {
+ // If we have a layer-group as the one or a parent, ensure it is listed as a valid layer.
+ needsAdd &= ( g_slist_find(const_cast<GSList *>(layers), curr) != NULL );
+ if ( (!(group->repr)) || (!(group->repr->parent())) ) {
+ needsAdd = false;
+ }
+ } else {
+ // If a non-layer group is a parent of layer groups, then show it also as a layer.
+ // TODO add the magic Inkscape group mode?
+ if ( group->repr && group->repr->parent() ) {
+ additional.insert(group);
+ } else {
+ needsAdd = false;
+ }
+ }
+ }
+ }
+ }
+ if ( needsAdd ) {
+ if ( !includes(layer) ) {
+ layersToAdd.insert(SP_GROUP(layer));
+ }
+ for ( std::set<SPGroup*>::iterator it = additional.begin(); it != additional.end(); ++it ) {
+ if ( !includes(*it) ) {
+ layersToAdd.insert(*it);
+ }
+ }
+ }
+ }
- for ( SPObject* curr = layer; curr && (curr != root) ; curr = SP_OBJECT_PARENT(curr) ) {
- if ( (curr != root) && root->isAncestorOf(curr) && !includes(curr) ) {
- // Filter out objects in the middle of being deleted
+ for ( std::set<SPGroup*>::iterator it = layersToAdd.begin(); it != layersToAdd.end(); ++it ) {
+ SPGroup* layer = *it;
+ // Filter out objects in the middle of being deleted
- // Such may have been the cause of bug 1339397.
- // See http://sourceforge.net/tracker/index.php?func=detail&aid=1339397&group_id=93438&atid=604306
+ // Such may have been the cause of bug 1339397.
+ // See http://sourceforge.net/tracker/index.php?func=detail&aid=1339397&group_id=93438&atid=604306
- SPObject const *higher = curr;
- while ( higher && (SP_OBJECT_PARENT(higher) != root) ) {
- higher = SP_OBJECT_PARENT(higher);
- }
- Node* node = higher ? SP_OBJECT_REPR(higher) : 0;
- if ( node && node->parent() ) {
- sigc::connection connection = curr->connectModified(sigc::mem_fun(*this, &LayerManager::_objectModified));
+ SPObject const *higher = layer;
+ while ( higher && (SP_OBJECT_PARENT(higher) != root) ) {
+ higher = SP_OBJECT_PARENT(higher);
+ }
+ Node* node = higher ? SP_OBJECT_REPR(higher) : 0;
+ if ( node && node->parent() ) {
+// Debug::EventTracker<DebugAddLayer> tracker(*layer);
- LayerWatcher *eye = new LayerWatcher(this, curr, connection);
- _watchers.push_back( eye );
- SP_OBJECT_REPR(curr)->addObserver(*eye);
+ sigc::connection connection = layer->connectModified(sigc::mem_fun(*this, &LayerManager::_objectModified));
- _addOne(curr);
- }
- }
+ LayerWatcher *eye = new LayerWatcher(this, layer, connection);
+ _watchers.push_back( eye );
+ SP_OBJECT_REPR(layer)->addObserver(*eye);
+
+ _addOne(layer);
}
}
}