Many changes are happening in the 7.x-2.x branch. At some point, we want to clean up the code. We should put it through a Coder review and fix items like:

* indent "break;" throughout the code to follow Drupal coding standards
* Review file organization in the module. We have stuff in includes/ and views/ for View support. Is that how we want to keep it?

This issue was originally posted in #1224766-30: Remove default media browser and replace with a default view.

Files: 
CommentFileSizeAuthor
#4 1293666-4.patch502 bytesstevector
PASSED: [[SimpleTest]]: [MySQL] 11 pass(es).
[ View ]

Comments

Dave Reid’s picture

Dave Reid’s picture

Assigned:Unassigned» Dave Reid
Devin Carlson’s picture

stevector’s picture

Status:Active» Needs review
StatusFileSize
new502 bytes
PASSED: [[SimpleTest]]: [MySQL] 11 pass(es).
[ View ]

Here's a start. Perhaps this should go in a different issue :)

ParisLiakos’s picture

Status:Needs review» Active

switching to correct status

Devin Carlson’s picture

Issue summary:View changes
Status:Active» Fixed

Marking this as fixed since the code referenced in the patch was removed with http://drupalcode.org/project/media.git/commit/4a9334f3489756acfe9330d7b... and much of the Media module has since been cleaned up.

I'd suggest filing separate issues for any further improvements you come across.

Status:Fixed» Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.