Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upcore-admin: separate xml loading #3123
Comments
marmarek
added this to the Release 4.0 milestone
Sep 26, 2017
marmarek
assigned
woju
Sep 26, 2017
marmarek
added
bug
C: core
P: major
labels
Sep 26, 2017
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
|
@woju do we still want this? |
andrewdavidwong
modified the milestones:
Release 4.0,
Release 4.0 updates
Mar 31, 2018
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
woju
Apr 18, 2018
Member
No, I don't think it is important. Currently things work, so don't fix if it ain't broken.
|
No, I don't think it is important. Currently things work, so don't fix if it ain't broken. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
woju
Apr 18, 2018
Member
This issue is abandoned, so I'm going to close it. If that's a mistake, feel free to reopen.
@marmarek @andrewdavidwong
|
This issue is abandoned, so I'm going to close it. If that's a mistake, feel free to reopen. |
woju
closed this
Apr 18, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
marmarek commentedSep 26, 2017
This was done when all objects kept reference to their respective XML object
for saving. This is no longer needed since there is no concurrent access to XML
from different tools anymore.
PropertyHolder.from_xml(), to be overloaded in children. Care should be takenwhen loading properties wrt loading order.