jhinkey/liferay-docs — pull request #126 by codyhoag (2014-04-09 20:15:16+00:00)
Title: Recycle Bin Edits

Hey Jim,

I've provided corrections/updates for **most** of your feedback. There are two of your comments I left because I'm unsure of an exact, detailed answer, so I've asked Julio about them and am waiting for a response.

In the meantime, I've provided commits to comply with your suggestions that are ready for your review. I'll provide the last of the edits once I have an answer from Julio.

One of your suggestions I wanted to explain a bit further is the following:

In **Implementing the Undo Functionality Step3: Display the Taglib**

You suggested on line [2071](https://github.com/jhinkey/liferay-docs/commit/32508c42aa1390f493402043b7617c63850c149a): **

In Julio's description of implementing the Undo button, this was one of his three steps because, in order to display the taglib for the Undo functionality specifically, we needed to provide information for the session messages so the session would know which elements were deleted. From what I understood, providing elements to the session were only required for the Undo functionality (since the Undo button appears simultaneous to the deletion of an entity), not for restoring entities from the Recycle Bin interface.

I outlined the `deleteSong` method in the *Moving Entries to the Recycle Bin* section, so I've already covered what this method does. To avoid confusion, I edited the code snippet to only display code related to the session messages in the Undo section (not the entire `deleteSong()` method), just so users know that is the highlighted code for the Undo functionality.

Let me know what you think.