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.

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


Dave Reid’s picture

Dave Reid’s picture

Assigned:Unassigned» Dave Reid
Devin Carlson’s picture

stevector’s picture

Status:Active» Needs review
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 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.