Convert View object to PSR-0

.

Just some small things i found.

If you review this patch please ignore the moved classes for the moment.

+++ b/lib/Drupal/views/View.phpundefined
@@ -2,21 +2,18 @@
+ * Definition of Drupal\views\view.

So i'm wondering whether it should be Drupal\views\View instead?

+++ b/lib/Drupal/views/View.phpundefined
@@ -2,21 +2,18 @@
-/**
- * @defgroup views_objects Objects that represent a View or part of a view
- * @{
- * These objects are the core of Views do the bulk of the direction and
- * storing of data. All database activity is in these objects.

Seems to be a bit odd to remove existing documentation.

+++ b/lib/Drupal/views/ViewsDbObject.phpundefined
@@ -0,0 +1,410 @@
+ * Definition of Drupal\views\views_db_object.

A bit odd as well...

+++ b/lib/Drupal/views/View.phpundefined
@@ -2,21 +2,18 @@
+use Drupal\views\ViewsDbObject;

don't need to do that when we are in the same namespace

.

This will conflict with https://drupal.org/node/1637552
Lets commit this (but don't forget to change the line from the previous issue)

+++ b/tests/views_analyze.testundefined
@@ -18,12 +18,12 @@ class ViewsAnalyzeTest extends ViewsSqlTest {
+    parent::setUp();
+//    module_enable(array('views_ui'));
     // @TODO Figure out why it's required to clear the cache here.
-    views_module_include('views_default', TRUE);
-    views_get_all_views(TRUE);
-    menu_router_rebuild();
+//    views_module_include('views_default', TRUE);
+//    views_get_all_views(TRUE);

Maybe you could explain that a bit, why this is required

+++ b/tests/views_upgrade.testundefined
@@ -175,7 +177,7 @@ class ViewsUpgradeTestCase extends ViewsSqlTest {
+      $view = new Drupal\views\View();

Just wondering why not use just new View();

+++ b/views.api.phpundefined
@@ -660,7 +660,7 @@ function hook_views_api() {
+  $view = new Drupal\views\View();

.

The last is correct, thats a api.php file.

Did some manual testing in the ui and still everything worked so lets get this in.

Huge thanks for this big converting patch!