cloudstrg 0.0.5 → 0.0.6

data/app/views/cloudstrg/remoteobjects/_form.html.erb

@@ -0,0 +1,33 @@
+<%= form_for(@remoteobject) do |f| %>
+  <% if @remoteobject.errors.any? %>
+    <div id="error_explanation">
+      <h2><%= pluralize(@remoteobject.errors.count, "error") %> prohibited this remoteobject from being saved:</h2>
+
+      <ul>
+      <% @remoteobject.errors.full_messages.each do |msg| %>
+        <li><%= msg %></li>
+      <% end %>
+      </ul>
+    </div>
+  <% end %>
+
+  <div class="field">
+    <%= f.label :user_id %><br />
+    <%= f.number_field :user_id %>
+  </div>
+  <div class="field">
+    <%= f.label :plugin_id %><br />
+    <%= f.number_field :plugin_id %>
+  </div>
+  <div class="field">
+    <%= f.label :filename %><br />
+    <%= f.text_field :filename %>
+  </div>
+  <div class="field">
+    <%= f.label :filehash %><br />
+    <%= f.text_field :filehash %>
+  </div>
+  <div class="actions">
+    <%= f.submit %>
+  </div>
+<% end %>

data/app/views/cloudstrg/remoteobjects/edit.html.erb

@@ -0,0 +1,6 @@
+<h1>Editing remoteobject</h1>
+
+<%= render 'form' %>
+
+<%= link_to 'Show', @remoteobject %> |
+<%= link_to 'Back', remoteobjects_path %>

data/app/views/cloudstrg/remoteobjects/index.html.erb

@@ -0,0 +1,29 @@
+<h1>Listing remoteobjects</h1>
+
+<table>
+  <tr>
+    <th>User</th>
+    <th>Plugin</th>
+    <th>Filename</th>
+    <th>Filehash</th>
+    <th></th>
+    <th></th>
+    <th></th>
+  </tr>
+
+<% @remoteobjects.each do |remoteobject| %>
+  <tr>
+    <td><%= remoteobject.user_id %></td>
+    <td><%= remoteobject.plugin_id %></td>
+    <td><%= remoteobject.filename %></td>
+    <td><%= remoteobject.filehash %></td>
+    <td><%= link_to 'Show', remoteobject %></td>
+    <td><%= link_to 'Edit', edit_remoteobject_path(remoteobject) %></td>
+    <td><%= link_to 'Destroy', remoteobject, method: :delete, data: { confirm: 'Are you sure?' } %></td>
+  </tr>
+<% end %>
+</table>
+
+<br />
+
+<%= link_to 'New Remoteobject', new_remoteobject_path %>

data/app/views/cloudstrg/remoteobjects/new.html.erb

@@ -0,0 +1,5 @@
+<h1>New remoteobject</h1>
+
+<%= render 'form' %>
+
+<%= link_to 'Back', remoteobjects_path %>

data/app/views/cloudstrg/remoteobjects/show.html.erb

@@ -0,0 +1,25 @@
+<p id="notice"><%= notice %></p>
+
+<p>
+  <b>User:</b>
+  <%= @remoteobject.user_id %>
+</p>
+
+<p>
+  <b>Plugin:</b>
+  <%= @remoteobject.plugin_id %>
+</p>
+
+<p>
+  <b>Filename:</b>
+  <%= @remoteobject.filename %>
+</p>
+
+<p>
+  <b>Filehash:</b>
+  <%= @remoteobject.filehash %>
+</p>
+
+
+<%= link_to 'Edit', edit_remoteobject_path(@remoteobject) %> |
+<%= link_to 'Back', remoteobjects_path %>

data/config/routes.rb

@@ -1,4 +1,10 @@
 Cloudstrg::Engine.routes.draw do
+  resources :configs
+
+  resources :remoteobjects
+
   resources :cloudstrgplugins
 
+  get '/cloudstrgplugins/:plugin_id/files(.:format)' => 'remoteobjects#index'
+
 end

data/config/routes.rb~

@@ -0,0 +1,8 @@
+Cloudstrg::Engine.routes.draw do
+  resources :remoteobjects
+
+  resources :cloudstrgplugins
+
+  get '/cloudstrgplugins/:plugin_id/files(.:format)' => 'remoteobjects#index'
+
+end

data/db/migrate/20121109112536_create_cloudstrg_remoteobjects.rb~

@@ -0,0 +1,12 @@
+class CreateCloudstrgRemoteobjects < ActiveRecord::Migration
+  def change
+    create_table :cloudstrg_remoteobjects do |t|
+      t.integer :user_id
+      t.integer :plugin_id
+      t.string :filename
+      t.fixnum :filehash
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20121109175154_create_cloudstrg_remoteobjects.rb

@@ -0,0 +1,12 @@
+class CreateCloudstrgRemoteobjects < ActiveRecord::Migration
+  def change
+    create_table :cloudstrg_remoteobjects do |t|
+      t.integer "#{Cloudstrg.user_class.downcase}_id".to_sym
+      t.integer :plugin_id
+      t.string :filename
+      t.string :filehash
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20121109175154_create_cloudstrg_remoteobjects.rb~

@@ -0,0 +1,12 @@
+class CreateCloudstrgRemoteobjects < ActiveRecord::Migration
+  def change
+    create_table :cloudstrg_remoteobjects do |t|
+      t.integer "#{Cloudstrg.user_class}_id".to_sym
+      t.integer :plugin_id
+      t.string :filename
+      t.string :filehash
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20121113105908_create_cloudstrg_configs.rb

@@ -0,0 +1,10 @@
+class CreateCloudstrgConfigs < ActiveRecord::Migration
+  def change
+    create_table :cloudstrg_configs do |t|
+      t.integer "#{Cloudstrg.user_class.downcase}_id".to_sym
+      t.integer :plugin_id
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20121113105908_create_cloudstrg_configs.rb~

@@ -0,0 +1,10 @@
+class CreateCloudstrgConfigs < ActiveRecord::Migration
+  def change
+    create_table :cloudstrg_configs do |t|
+      t.integer "#{Cloudstrg.user_class}_id".to_sym
+      t.integer :plugin_id
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20121116111055_rename_cloudstrg_config_plugin_id_to_cloudstrg_config_cloudstrgplugin_id.rb

@@ -0,0 +1,5 @@
+class RenameCloudstrgConfigPluginIdToCloudstrgConfigCloudstrgpluginId < ActiveRecord::Migration
+  def change
+    rename_column :cloudstrg_configs, :plugin_id, :cloudstrgplugin_id
+  end
+end

data/db/migrate/20121119105425_rename_remoteobject_plugin_id_to_remoteobject_cloudstrgplugin_id.rb

@@ -0,0 +1,5 @@
+class RenameCloudstrgRemoteobjectPluginIdToCloudstrgRemoteobjectCloudstrgpluginId < ActiveRecord::Migration
+  def change
+    rename_column :cloudstrg_remoteobjects, :plugin_id, :cloudstrgplugin_id
+  end
+end

data/db/migrate/20121119184135_add_file_remote_id_to_cloudstrg_remoteobjects.rb

@@ -0,0 +1,5 @@
+class AddFileRemoteIdToCloudstrgRemoteobjects < ActiveRecord::Migration
+  def change
+    add_column :cloudstrg_remoteobjects, :file_remote_id, :integer
+  end
+end

data/db/migrate/20121119184135_add_file_remote_id_to_cloudstrg_remoteobjects.rb~

@@ -0,0 +1,5 @@
+class AddFileRemoteIdToCloudstrgRemoteobjects < ActiveRecord::Migration
+  def change
+    add_column :cloudstrg_remoteobjects, :file_remote_id, :string
+  end
+end

data/db/migrate/20121120022309_change_file_remote_id_type_to_string.rb

@@ -0,0 +1,6 @@
+class ChangeFileRemoteIdTypeToString < ActiveRecord::Migration
+  def change
+    change_column :cloudstrg_remoteobjects, :file_remote_id, :string
+
+  end
+end

data/lib/cloudstrg/cloudstrg.rb

@@ -4,13 +4,13 @@ module CloudStrg
   # This method returns an instance of an specified driver
   #
   # Params: the "params" variable must contain the following fields
-  #   type: the name of the selected driver.
+  #   plugin_id: the id of the selected driver.
   #   
   # Returns: 
   #   An instance of the specified driver.
   #
   def self.new_driver params
-    type = params[:type]
+    type = Cloudstrg::Cloudstrgplugin.find(params[:plugin_id]).plugin_name
     require "#{type}strg/#{type}strg"
     return Kernel.const_get("#{type.capitalize}Strg").new(params)
   end
@@ -20,7 +20,8 @@ module CloudStrg
   # This method returns a list of the available storage drivers.
   #
   def self.driver_list
-    l = Cloudstrg::Cloudstrgplugin.select('plugin_name')
+    l = []
+    Cloudstrg::Cloudstrgplugin.find(:all).each {|plugin| l << [plugin.plugin_name, plugin.id]}
     l.sort!
     return l
   end
@@ -35,9 +36,9 @@ module CloudStrg
     # This method performs the previous configuration that the current driver needs to work.
     #
     # Params: the "params" variable must contain the following fields
-    #   redirect: the name of the selected driver,
-    #   username: the name of the selected driver,
-    #   session: the name of the selected driver.
+    #   redirect: url that we want the application to redirect us before accepting the selected cloud engine terms,
+    #   user: the current user,
+    #   session: the session hash.
     #   
     # Returns: 
     #   This method returns two parameters:
@@ -57,7 +58,7 @@ module CloudStrg
     #   file_content: the content of the file.
     #   
     # Returns: 
-    #   This method returns true if the operation success, otherwise it returns false.
+    #   This method returns the local id of the remote object if the operation success, otherwise it returns false.
     #
     def create_file params
       raise NotImplementedError
@@ -92,7 +93,7 @@ module CloudStrg
     #   file_content: the content of the file.
     #   
     # Returns: 
-    #   This method returns true if the operation success, otherwise it returns false.
+    #   This method returns the local id of the remote object if the operation success, otherwise it returns false.
     #
     def update_file params
       raise NotImplementedError
@@ -118,26 +119,46 @@ module CloudStrg
       raise NotImplementedError
     end
 
-    def save_hash(username, filename, filecontent)
-      plugin_name = self.class.to_s.split('Strg')[0].downcase
+    def save_remoteobject(user, filename, filecontent, file_remote_id)
+      #plugin_name = self.class.to_s.split('Strg')[0].downcase
       
-      if Cloudstrgfile.where(:username => username, :strgdriver => plugin_name, :filename => filename).size > 0
-        Cloudstrgfile.delete_all(:username => username, :strgdriver => plugin_name, :filename => filename)
+      file = Cloudstrg::Remoteobject.where(:user_id => user, :cloudstrgplugin_id => user.cloudstrgconfig.cloudstrgplugin, :filename => filename)
+      if not file.empty?
+        file = file[0]
+        file.filehash = filecontent.hash.to_s
+      else
+        file = user.cloudstrgconfig.cloudstrgplugin.remoteobjects.build(:user_id => user, :filename => filename, :filehash => filecontent.hash.to_s, :file_remote_id => file_remote_id)
       end
-      Cloudstrgfile.create :username => username, :filename => filename, :filehash => filecontent.hash.to_s, :strgdriver => plugin_name
-      true
+      file.save
+      return file
     end
 
-    def check_hash(username, filename, filecontent)
-      plugin_name = self.class.to_s.split('Strg')[0].downcase
-      
-      l = Cloudstrgfile.where(:username => username, :strgdriver => plugin_name, :filename => filename)
-      if l.size != 1
-        return l[0].filehash == filecontent.hash.to_s
-      else
-        false
-      end
+    def get_remoteobject(ro_id)
+      return Cloudstrg::Remoteobject.find(ro_id)
+    end 
+
+    def delete_remoteobject(ro_id)
+      return Cloudstrg::Remoteobject.find(ro_id).destroy
     end
 
+#    def check_hash(username, filename, filecontent)
+#      plugin_name = self.class.to_s.split('Strg')[0].downcase
+#      
+#      l = Cloudstrg::Remoteobject.where(:username => username, :strgdriver => plugin_name, :filename => filename)
+#      if l.size == 1
+#        return l[0].filehash == filecontent.hash.to_s
+#      else
+#        false
+#      end
+#    end
+
   end
+
+  class RONotConfigured < Exception
+  end
+
+  class ROValidationRequired < Exception
+  end
+
+
 end

data/lib/cloudstrg/cloudstrg.rb~

@@ -0,0 +1,164 @@
+module CloudStrg
+  
+  ##
+  # This method returns an instance of an specified driver
+  #
+  # Params: the "params" variable must contain the following fields
+  #   plugin_id: the id of the selected driver.
+  #   
+  # Returns: 
+  #   An instance of the specified driver.
+  #
+  def self.new_driver params
+    type = Cloudstrg::Cloudstrgplugin.find(params[:plugin_id]).plugin_name
+    require "#{type}strg/#{type}strg"
+    return Kernel.const_get("#{type.capitalize}Strg").new(params)
+  end
+
+  
+  ##
+  # This method returns a list of the available storage drivers.
+  #
+  def self.driver_list
+    l = []
+    Cloudstrg::Cloudstrgplugin.find(:all).each {|plugin| l << [plugin.plugin_name, plugin.id]}
+    l.sort!
+    return l
+  end
+
+  ###
+  # This class must be inherited by every single driver definition in order to preserve 
+  # the coherence between them.
+  #
+  class CloudStorage
+  
+    ###
+    # This method performs the previous configuration that the current driver needs to work.
+    #
+    # Params: the "params" variable must contain the following fields
+    #   redirect: url that we want the application to redirect us before accepting the selected cloud engine terms,
+    #   user: the current user,
+    #   session: the session hash.
+    #   
+    # Returns: 
+    #   This method returns two parameters:
+    #     session: the session variable with the new parameters added,
+    #     uri: the uri where the user will allow the application to use the desired service, 
+    #       or false if the configuration was completed successfully.
+    #
+    def config params
+      raise NotImplementedError
+    end
+  
+    ###
+    # This method performs the creation of a file.
+    #
+    # Params: the "params" variable must contain the following fields
+    #   filename: the name of the file to create,
+    #   file_content: the content of the file.
+    #   
+    # Returns: 
+    #   This method returns the local id of the remote object if the operation success, otherwise it returns false.
+    #
+    def create_file params
+      raise NotImplementedError
+    end
+
+    #def create_folder params
+    #  raise NotImplementedError
+    #end
+
+    ###
+    # This method performs the request to obtain a desired file content
+    #
+    # Params: the "params" variable must contain the following fields
+    #   fileid: the id of the selected file.
+    #   
+    # Returns: 
+    #   This method returns three parameters:
+    #     filename: the name of the file,
+    #     fileid: the id of the file, 
+    #     file_content: the content of the file.
+    #
+    def get_file params
+      raise NotImplementedError
+    end
+  
+    ###
+    # This method performs the update of a file.
+    #
+    # Params: the "params" variable must contain the following fields
+    #   filename: the name of the file to edit,
+    #   fileid: the id of the file, 
+    #   file_content: the content of the file.
+    #   
+    # Returns: 
+    #   This method returns the local id of the remote object if the operation success, otherwise it returns false.
+    #
+    def update_file params
+      raise NotImplementedError
+    end
+
+    ###
+    # This method performs the remove of a file.
+    #
+    # Params: the "params" variable must contain the following fields
+    #   fileid: the id of the file.
+    #
+    def remove_file params
+      raise NotImplementedError
+    end
+
+    ###
+    # This method returns a list of the available files created by the application.
+    #   
+    # Returns: 
+    #   This method returns a list of tuples, containing: [[filename, fileid], ...] 
+    #
+    def list_files
+      raise NotImplementedError
+    end
+
+    def save_remoteobject(user, filename, filecontent, file_remote_id)
+      #plugin_name = self.class.to_s.split('Strg')[0].downcase
+      
+      file = Cloudstrg::Remoteobject.where(:user_id => user, :cloudstrgplugin_id => user.cloudstrgconfig.cloudstrgplugin, :filename => filename)
+      if not file.empty?
+        file = file[0]
+        file.filehash = filecontent.hash.to_s
+      else
+        file = user.cloudstrgconfig.cloudstrgplugin.remoteobjects.build(:user_id => user, :filename => filename, :filehash => filecontent.hash.to_s, :file_remote_id => file_remote_id)
+      end
+      file.save
+      return file
+    end
+
+    def get_remoteobject(ro_id)
+      return Cloudstrg::Remoteobject.find(ro_id)
+    end 
+
+    def delete_remoteobject(ro_id)
+      return Cloudstrg::Remoteobject.find(ro_id).destroy
+    end
+
+#    def check_hash(username, filename, filecontent)
+#      plugin_name = self.class.to_s.split('Strg')[0].downcase
+#      
+#      l = Cloudstrg::Remoteobject.where(:username => username, :strgdriver => plugin_name, :filename => filename)
+#      if l.size == 1
+#        return l[0].filehash == filecontent.hash.to_s
+#      else
+#        false
+#      end
+#    end
+
+  end
+
+  class RONotConfigured < Exception
+  end
+
+  class ROValidationRequired < Exception
+  end
+
+
+end

data/lib/cloudstrg/version.rb

@@ -1,3 +1,3 @@
 module Cloudstrg
-  VERSION = "0.0.5"
+  VERSION = "0.0.6"
 end

data/lib/cloudstrg.rb

@@ -1,4 +1,5 @@
 require "cloudstrg/engine"
 
 module Cloudstrg
+  mattr_accessor :user_class
 end

data/test/dummy/db/migrate/20121107132518_create_cloudstrg_cloudstrgplugins.cloudstrg.rb

@@ -0,0 +1,11 @@
+# This migration comes from cloudstrg (originally 20121105173015)
+class CreateCloudstrgCloudstrgplugins < ActiveRecord::Migration
+  def change
+    create_table :cloudstrg_cloudstrgplugins do |t|
+      t.string :plugin_name
+      t.string :version
+
+      t.timestamps
+    end
+  end
+end

data/test/dummy/db/schema.rb

@@ -0,0 +1,23 @@
+# encoding: UTF-8
+# This file is auto-generated from the current state of the database. Instead
+# of editing this file, please use the migrations feature of Active Record to
+# incrementally modify your database, and then regenerate this schema definition.
+#
+# Note that this schema.rb definition is the authoritative source for your
+# database schema. If you need to create the application database on another
+# system, you should be using db:schema:load, not running all the migrations
+# from scratch. The latter is a flawed and unsustainable approach (the more migrations
+# you'll amass, the slower it'll run and the greater likelihood for issues).
+#
+# It's strongly recommended to check this file into your version control system.
+
+ActiveRecord::Schema.define(:version => 20121107132518) do
+
+  create_table "cloudstrg_cloudstrgplugins", :force => true do |t|
+    t.string   "plugin_name"
+    t.string   "version"
+    t.datetime "created_at",  :null => false
+    t.datetime "updated_at",  :null => false
+  end
+
+end

data/test/dummy/log/development.log

@@ -1,3 +1,19 @@
 Connecting to database specified by database.yml
 Connecting to database specified by database.yml
 Connecting to database specified by database.yml
+Connecting to database specified by database.yml
+Connecting to database specified by database.yml
+   (0.2ms)  select sqlite_version(*)
+   (139.8ms)  CREATE TABLE "schema_migrations" ("version" varchar(255) NOT NULL) 
+   (0.1ms)  PRAGMA index_list("schema_migrations")
+   (101.2ms)  CREATE UNIQUE INDEX "unique_schema_migrations" ON "schema_migrations" ("version")
+   (0.2ms)  SELECT "schema_migrations"."version" FROM "schema_migrations" 
+Migrating to CreateCloudstrgCloudstrgplugins (20121107132518)
+   (0.1ms)  begin transaction
+   (1.1ms)  CREATE TABLE "cloudstrg_cloudstrgplugins" ("id" INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL, "plugin_name" varchar(255), "version" varchar(255), "created_at" datetime NOT NULL, "updated_at" datetime NOT NULL) 
+   (0.3ms)  INSERT INTO "schema_migrations" ("version") VALUES ('20121107132518')
+   (132.9ms)  commit transaction
+   (0.7ms)  select sqlite_version(*)
+   (0.3ms)  SELECT "schema_migrations"."version" FROM "schema_migrations" 
+   (0.1ms)  PRAGMA index_list("cloudstrg_cloudstrgplugins")
+Connecting to database specified by database.yml

data/test/fixtures/cloudstrg/configs.yml

@@ -0,0 +1,9 @@
+# Read about fixtures at http://api.rubyonrails.org/classes/ActiveRecord/Fixtures.html
+
+one:
+  user_id: 1
+  plugin_id: 1
+
+two:
+  user_id: 1
+  plugin_id: 1

data/test/fixtures/cloudstrg/remoteobjects.yml

@@ -0,0 +1,13 @@
+# Read about fixtures at http://api.rubyonrails.org/classes/ActiveRecord/Fixtures.html
+
+one:
+  user_id: 1
+  plugin_id: 1
+  filename: MyString
+  filehash: MyString
+
+two:
+  user_id: 1
+  plugin_id: 1
+  filename: MyString
+  filehash: MyString

data/test/functional/cloudstrg/configs_controller_test.rb

@@ -0,0 +1,51 @@
+require 'test_helper'
+
+module Cloudstrg
+  class ConfigsControllerTest < ActionController::TestCase
+    setup do
+      @config = configs(:one)
+    end
+  
+    test "should get index" do
+      get :index
+      assert_response :success
+      assert_not_nil assigns(:configs)
+    end
+  
+    test "should get new" do
+      get :new
+      assert_response :success
+    end
+  
+    test "should create config" do
+      assert_difference('Config.count') do
+        post :create, config: { plugin_id: @config.plugin_id, user_id: @config.user_id }
+      end
+  
+      assert_redirected_to config_path(assigns(:config))
+    end
+  
+    test "should show config" do
+      get :show, id: @config
+      assert_response :success
+    end
+  
+    test "should get edit" do
+      get :edit, id: @config
+      assert_response :success
+    end
+  
+    test "should update config" do
+      put :update, id: @config, config: { plugin_id: @config.plugin_id, user_id: @config.user_id }
+      assert_redirected_to config_path(assigns(:config))
+    end
+  
+    test "should destroy config" do
+      assert_difference('Config.count', -1) do
+        delete :destroy, id: @config
+      end
+  
+      assert_redirected_to configs_path
+    end
+  end
+end