Coding style issues

While you're on it please look at my list

+++ b/core/modules/project_browser/js/project_browser_categories_widget.jsundefined
@@ -0,0 +1,11 @@
+(function ($) {
+  $('#edit-categories').multiselect({
+    noneSelectedText: '" . t('Choose') . "...',

If I'm not mistaken there is a new rule to add "strict" in every core js file. Just look at other js files.

+++ b/core/modules/project_browser/js/project_browser_more_link.jsundefined
@@ -0,0 +1,32 @@
+(function ($) {
+  $(document).ready(function() {
+    /**
+     * The project descriptions are by default trimmed to a certain height. When the user

Shouldn't we use drupal behaviours in here?

+++ b/core/modules/project_browser/project_browser.admin.incundefined
@@ -0,0 +1,36 @@
+
+// ======================================
+// Administration Page:
+// ======================================

Remove this we don't do this in core.

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+<?php
+/**

Add a newline after "<?php"

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+// Include the classes from the 'update' module

This comment isn't needed and is different from normal core behavior. Let's remove it

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+ * no real processing is done, we just redirect to the install/select_versions page

Exceeds 80 chars

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+ * Builds the filters form

Yeah we need "." on the end of these sentences like said before

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+      // Set the direction of the sort link to the opposite of what it currently is

Exceeds 80 chars

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+ *   (Optional) Set this to TRUE if you want to get all of the supported sort methods. Defaults to FALSE

Exceeds 80 chars

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+        // Create a new key so that there are no duplicate categories from different sites

Exceeds 80 chars

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+// ======================================
+// Server Related Functions:

Remove this block. Not a core pattern

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+ *   'text' => 'views', // The text that was entered as the search query, or '' if none
+ *   'categories' => array() // The categories that were selected, if any
+ *   'type' => 'module', // The type of project being searched
+ *   'page' => 3, // The zero-based page number

Thie is hard to read. Is it possible to format this a bit better?

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+ *   Returns an associative array of servers, populated from the project_browser_servers variable,

Exceeds 80 chars

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+// ======================================
+// Batch Operations:

Again: I would remove this.

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+ * Helper function to download a project. This code is mostly copied and pasted from
+ * modules/update/update.manager.inc

Don't use "Helper function". Just start with a 3th person verb in your short summary. Should fit on one line! (less than 80 chars)

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+    // Add this to the session variable and remove it from the install_queue variable

Exceeds 80 chars

+++ b/core/modules/project_browser/project_browser.incundefined
@@ -0,0 +1,993 @@
+ * If there were any errors, they are reported here with drupal_set_message(). The
+ * user is then redirected to the select versions page if there were errors, the

Exceeds 80 chars

+++ b/core/modules/project_browser/project_browser.moduleundefined
@@ -0,0 +1,485 @@
+// ======================================
+// Theme:

Let's remove this ;)

+++ b/core/modules/project_browser/project_browser.moduleundefined
@@ -0,0 +1,485 @@
+ * Add some variables for the project browser block theme

AddS

There are more errors but with my comments you should be able to fix them all. If you're unsure, read node/1354 on drupal.org