Updated user id & project id fetching

Removed pass by reference of the project files. Made the method look for
the request fields `userId` & `projectId` instead of the txt files that
were user until now.

Author: fpapadopou

Symfony/src/Codebender/BuilderBundle/Controller/DefaultController.php

@@ -81,9 +81,9 @@ protected function compile($contents)
     {
         $apiHandler = $this->get('codebender_builder.handler');
 
-        $files = $contents["files"];
+        $contents = $this->checkForUserIdProjectId($contents);
 
-        $this->checkForUserIdProjectId($files);
+        $files = $contents["files"];
 
         $userLibraries = array();
 
@@ -199,28 +199,20 @@ protected function returnProvidedAndFetchedLibraries($projectFiles, $userLibrari
       * Checks if project id and user id txt files exist in the request files.
       * If not, creates these files with null id
       *
-      * @param array $projectFiles
+      * @param array $requestContents
+      * @return array
       */
-    protected function checkForUserIdProjectId(&$projectFiles)
+    protected function checkForUserIdProjectId($requestContents)
     {
-        $foundProject = false;
-        $foundUser = false;
-
-        foreach ($projectFiles as $file) {
-            if (preg_match('/(?<=user_)[\d]+/', $file['filename'])) {
-                $foundUser = true;
-            }
-            if (preg_match('/(?<=project_)[\d]+/', $file['filename'])) {
-                $foundProject = true;
-            }
+        if (!array_key_exists('userId', $requestContents)) {
+            $requestContents['userId'] = 'null';
         }
 
-        if (!$foundUser) {
-            $projectFiles[] = array('filename' => 'user_null.txt', 'content' => '');
-        }
-        if (!$foundProject) {
-            $projectFiles[] = array('filename' => 'project_null.txt', 'content' => '');
+        if (!array_key_exists('projectId', $requestContents)) {
+            $requestContents['projectId'] = 'null';
         }
+
+        return $requestContents;
     }
 }