#61 almost done

app/controllers/magazines_controller.rb

@@ -85,12 +85,32 @@ def check_for_cancel
 
   # Request joining a magazine
   def join
-    joinh = { "user_id" => current_user.id, "magazine_id" => params[:id] }
+    joinh = { 'user_id' => current_user.id, 'magazine_id' => params[:id] }
     Requestjoiningmagazine.create(joinh)
     @magazine = Magazine.find(params[:id])
     redirect_to @magazine
   end
 
+  def showrequests
+    @request = Requestjoiningmagazine.where('magazine_id = ?', params[:id])
+    @users = []
+    @request.each do |r|
+      @users.push(User.find(r.user_id))
+    end
+  end
+
+  def approverequest
+    @req = Requestjoiningmagazine.where("user_id = ? AND magazine_id = ?", params[:user], params[:id])
+    @req.destroy_all
+    joinh = { 'user_id' => params[:user], 'magazine_id' => params[:id] }
+    Requestjoiningmagazine.destroy_all(joinh)
+
+    @magazine = Magazine.find(params[:id])
+    @user = User.find(params[:user])
+    @magazine.users << @user
+    redirect_to :action => :showrequests
+  end
+
   private
 
   # Use callbacks to share common setup or constraints between actions.

app/models/ability.rb

@@ -15,6 +15,8 @@ def initialize(user)
       can :create, Magazine
       can :update, Magazine
       can :join, Magazine
+      can :showrequests, Magazine
+      can :approverequest, Magazine
       can :update, Comment do |c|
         c.user_id == user.id
       end

app/models/magazine.rb

@@ -2,8 +2,6 @@
 # 5.4.2015
 # Magazines model
 class Magazine < ActiveRecord::Base
-  has_many :joinmagazines
-  has_many :magazines, through: :joinmagazines
   # many-to-many relationship between magazine
   # requested to join to by many users
   has_many :requestjoiningmagazines

app/models/user.rb

@@ -1,8 +1,5 @@
 # User Model
 class User < ActiveRecord::Base
-  has_many :joinmagazines
-  has_many :magazines, through: :joinmagazines
-
   # many-to-many relationship between user requests joining magazine
   has_many :requestjoiningmagazines
   has_many :magazines, through: :requestjoiningmagazines

app/views/magazines/show.html.erb

@@ -20,4 +20,4 @@
 <%= link_to 'Back', magazines_path %> |
 <! link to request joining magazine>
 <%= link_to 'Join this magazine', action: "join" %> |
-<%= link_to 'View joining requests', {:controller => 'magazinesjoin', :action => 'index', :magazine => @magazine.id }%>
+<%= link_to 'View joining requests', action: 'showrequests'%>

app/views/magazines/showrequests.html.erb

@@ -0,0 +1,37 @@
+<div class="editor" >
+<h1>Joining requests</h1>
+
+   <div class="row">
+    <div class="col-sm-3 ">
+    Name
+    </div>
+       <div class="col-sm-3 ">
+    email
+    </div>
+   </div>
+
+    <% @users.each do |user| %>
+
+      <div class="row">
+      <div class="col-sm-3 ">
+      <%= user.firstname %>  &nbsp;<%= user.lastname %>
+      </div>
+      <div class="col-sm-3 ">
+      <%= user.email %>
+      </div>
+
+      <div class="col-sm-3 ">
+       <%= link_to 'Approve', {:action => "approverequest", :user => user.id } %>
+       </div>
+
+       <div class="col-sm-3 ">
+       <%= link_to 'Reject', sponsors_path(@s,:type =>"Reject") %>
+       </div>
+       </div>
+   <%end%>
+  <% if @users.size == 0 %>
+    <em>No Joining requests are found</em>
+  <% end %>
+
+
+</div>

config/routes.rb

@@ -1,11 +1,20 @@
 Rails.application.routes.draw do
   resources :magazines
-  resources :magazinesjoin
   resources :magazines do
     member do
       get 'join'
     end
   end
+  resources :magazines do
+    member do
+      get 'showrequests'
+    end
+  end
+  resources :magazines do
+    member do
+      get 'approverequest'
+    end
+  end
 
   # adding resources
   resources :sponsors

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20150407103735) do
+ActiveRecord::Schema.define(version: 20150406201640) do
 
   create_table "articles", force: :cascade do |t|
     t.string   "title"
@@ -31,16 +31,6 @@
     t.datetime "updated_at", null: false
   end
 
-  create_table "joinmagazines", force: :cascade do |t|
-    t.integer  "user_id"
-    t.integer  "magazine_id"
-    t.datetime "created_at",  null: false
-    t.datetime "updated_at",  null: false
-  end
-
-  add_index "joinmagazines", ["magazine_id"], name: "index_joinmagazines_on_magazine_id"
-  add_index "joinmagazines", ["user_id"], name: "index_joinmagazines_on_user_id"
-
   create_table "magazines", force: :cascade do |t|
     t.string   "name"
     t.text     "decription"