René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Do not allow top_level to be a child of another subclass","articleBody":"### Environment\r\n* DiffSync version:  \u003c!-- Example: 1.0.0 --\u003e\r\n2.1\r\n\r\n\u003c!--\r\n    Describe in detail the new functionality you are proposing.\r\n--\u003e\r\n### Proposed Functionality\r\nDo not allow `top_level` models to be a child of another subclass.\r\n\r\nFor example, when:\r\n\r\n```\r\nclass Site(DiffSyncModel):\r\n    _modelname = \"site\"\r\n    _identifiers = (\"resource_id\",)\r\n\r\nclass Location(DiffSyncModel):\r\n    _modelname = \"location\"\r\n    _identifiers = (\"resource_id\",)\r\n    _children = {\"site\": \"sites\"}\r\n```\r\n\r\nAnd the DiffSync implementation includes:\r\n```\r\nclass MyDiffSync(DiffSync):\r\n    top_level = [\"location\", \"site\"]\r\n\r\n    location = Location\r\n    site = Site\r\n```\r\n\r\nThis configuration causes duplication of site models during diff and sync operations, as they are treated both as independent top-level models and as children of location.\r\n\r\n\u003c!-- Convey an example use case for your proposed feature. Write from the perspective of a user who would benefit from the proposed functionality and describe how. --\u003e\r\n\r\nUse Case\r\nUsers who define complex hierarchies in their DiffSync models would benefit from this feature by avoiding duplication in diff and sync results. This would improve data consistency and ensure that models have clear and non-overlapping roles.\r\n\r\nFor example, if Site is both a top-level model and a child of Location, performing a diff might result in Site objects being reported twice: once as part of the top-level site model and again as part of the location hierarchy. Disallowing this configuration ensures clean and predictable behavior for synchronization processes.\r\n","author":{"url":"https://github.com/daveshm","@type":"Person","name":"daveshm"},"datePublished":"2024-12-11T06:00:54.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/285/diffsync/issues/285"}