Task #40754

Story #41101: As user, I want a well-working content editing (Bugfix Story)

re-implement delete node

Added by Rens Admiraal almost 3 years ago. Updated almost 3 years ago.

Status:Resolved Start date:
Priority:Must have Due date:
Assigned To:- % Done:

100%

Category:Content Editing Spent time: -
Target version:-

Related issues

related to Base Distribution - Task #41315: Fix deletion of nodes Closed

Associated revisions

Revision 7b970f81
Added by Robert Lemke almost 3 years ago

[BUGFIX] Fix setRemoved() for proxy nodes

The method Node::setRemoved() has not been implemented for ProxyNode
and thus nodes which are proxy nodes, are not marked as removed when
using that method.

This patch also introduces tests for that part and a new LoggingAspect
which - for now - only logs removal of nodes.

This bug has been introduced in 773160cbd25847049a8b57a35a3e3cfb96651be1

Change-Id: I903e144d037de9698579ee68c634a517f0f0eaa0
Related: #40754

Revision 2ed5933d
Added by Robert Lemke almost 3 years ago

[BUGFIX] Re-implement deletion of content elements

This adds the ability to remove content elements through the content
module.

Change-Id: I634b3a208df2e8edcc669531ec71e31ab2b9a502
Resolves: #40754

History

#1 Updated by Sebastian Kurfuerst almost 3 years ago

  • Assigned To set to Sebastian Kurfuerst

#2 Updated by Sebastian Kurfuerst almost 3 years ago

  • Status changed from New to Accepted

#3 Updated by Sebastian Kurfuerst almost 3 years ago

  • Status changed from Accepted to New

#4 Updated by Sebastian Kurfuerst almost 3 years ago

  • Status changed from New to Accepted

#5 Updated by Gerrit Code Review almost 3 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14519

#7 Updated by Sebastian Kurfuerst almost 3 years ago

  • Status changed from Accepted to Under Review

#8 Updated by Robert Lemke almost 3 years ago

  • Category set to Content Editing
  • Priority changed from Should have to Must have
  • Parent task changed from #40628 to #41311

#9 Updated by Robert Lemke almost 3 years ago

  • Assigned To changed from Sebastian Kurfuerst to Robert Lemke

#10 Updated by Robert Lemke almost 3 years ago

  • Status changed from Under Review to Accepted

#11 Updated by Robert Lemke almost 3 years ago

  • Status changed from Accepted to New
  • Assigned To deleted (Robert Lemke)
  • Target version changed from Sprint 10 (1.0 TP1) to Sprint February 2013
  • Parent task changed from #41311 to #41101

#12 Updated by Gerrit Code Review almost 3 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15047

#13 Updated by Gerrit Code Review almost 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15047

#14 Updated by Robert Lemke almost 3 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset commit:2ed5933d9fdeaae3c5bad9040883b2fa53844b97.

Also available in: Atom PDF