[MAGNOLIA-5426] DumperUtil should offer a dump method with a single Node argument Created: 28/Oct/13  Updated: 18/Dec/13  Resolved: 17/Dec/13

Status: Closed
Project: Magnolia
Component/s: core
Affects Version/s: 4.5.12, 5.0
Fix Version/s: 4.5.15, 5.2.1

Type: Improvement Priority: Neutral
Reporter: Nils Breunese Assignee: Milan Divilek
Resolution: Fixed Votes: 0
Labels: maintenance, vpro
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Date of First Response:

 Description   

It would be nice if DumperUtil had a dump(Node node) method, just like it has a dump(Content content) method.



 Comments   
Comment by Roman Kovařík [ 13/Dec/13 ]

Please mark old methods with Content parameter as deprecated since ... .

Comment by Milan Divilek [ 16/Dec/13 ]

Reopen:
info.magnolia.cms.util.DumperUtil#dump(javax.jcr.Node, int)
info.magnolia.cms.util.DumperUtil#dump(info.magnolia.cms.core.Content, int)

We have same code in all of those methods. Instead of have same code in all of them we should delegate to one of them.

Comment by Milan Divilek [ 16/Dec/13 ]

Reopen:
info.magnolia.cms.util.DumperUtil#dump(info.magnolia.cms.core.Content, int, java.io.PrintStream) was marked as deprecated with #dump(Node, int, PrintWriter) which is not true. Instead we should add #dump(javax.jcr.Node, int, java.io.PrintStream) method to avoid using deprecated content API.

Generated at Mon Feb 12 04:05:03 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.