diff --git a/Gemfile b/Gemfile index c1f1ee3e2..b268c825d 100644 --- a/Gemfile +++ b/Gemfile @@ -17,7 +17,7 @@ gem 'ffaker' # Bundle edge Rails instead: gem 'rails', github: 'rails/rails' gem 'rails', '~> 5.1.4' # Use sqlite3 as the database for Active Record -gem 'sqlite3' + # Use Puma as the app server gem 'puma', '~> 3.7' # Use SCSS for stylesheets @@ -37,9 +37,16 @@ gem 'jbuilder', '~> 2.5' # gem 'redis', '~> 3.0' # Use ActiveModel has_secure_password # gem 'bcrypt', '~> 3.1.7' - +gem 'bootstrap-sass', '~>3.3.7' # Use Capistrano for deployment # gem 'capistrano-rails', group: :development +gem 'jquery-rails' + +gem 'kaminari' + +group :production do + gem 'pg' +end group :development, :test do # Call 'byebug' anywhere in the code to stop execution and get a debugger console @@ -51,6 +58,8 @@ group :development, :test do gem 'factory_bot_rails' gem 'shoulda-matchers', '~> 3.1' gem 'rails-controller-testing' + gem 'sqlite3' + end group :development do diff --git a/Gemfile.lock b/Gemfile.lock index e55e9522f..e695f18de 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -41,11 +41,16 @@ GEM addressable (2.5.2) public_suffix (>= 2.0.2, < 4.0) arel (8.0.0) + autoprefixer-rails (8.6.5) + execjs bcrypt (3.1.11) bcrypt (3.1.11-java) bcrypt (3.1.11-x64-mingw32) bcrypt (3.1.11-x86-mingw32) bindex (0.5.0) + bootstrap-sass (3.3.7) + autoprefixer-rails (>= 5.2.1) + sass (>= 3.3.4) builder (3.2.3) byebug (10.0.0) capybara (2.17.0) @@ -97,6 +102,22 @@ GEM jbuilder (2.7.0) activesupport (>= 4.2.0) multi_json (>= 1.2) + jquery-rails (4.3.3) + rails-dom-testing (>= 1, < 3) + railties (>= 4.2.0) + thor (>= 0.14, < 2.0) + kaminari (1.1.1) + activesupport (>= 4.1.0) + kaminari-actionview (= 1.1.1) + kaminari-activerecord (= 1.1.1) + kaminari-core (= 1.1.1) + kaminari-actionview (1.1.1) + actionview + kaminari-core (= 1.1.1) + kaminari-activerecord (1.1.1) + activerecord + kaminari-core (= 1.1.1) + kaminari-core (1.1.1) listen (3.1.5) rb-fsevent (~> 0.9, >= 0.9.4) rb-inotify (~> 0.9, >= 0.9.7) @@ -124,6 +145,9 @@ GEM nokogiri (1.8.1-x86-mingw32) mini_portile2 (~> 2.3.0) orm_adapter (0.5.0) + pg (1.0.0) + pg (1.0.0-x64-mingw32) + pg (1.0.0-x86-mingw32) public_suffix (3.0.1) puma (3.11.2) puma (3.11.2-java) @@ -251,6 +275,7 @@ PLATFORMS x86-mswin32 DEPENDENCIES + bootstrap-sass (~> 3.3.7) byebug capybara (~> 2.13) carrierwave @@ -259,7 +284,10 @@ DEPENDENCIES factory_bot_rails ffaker jbuilder (~> 2.5) + jquery-rails + kaminari listen (>= 3.0.5, < 3.2) + pg puma (~> 3.7) rails (~> 5.1.4) rails-controller-testing diff --git a/app/assets/javascripts/application.js b/app/assets/javascripts/application.js index 46b20359f..8ce53dfbc 100644 --- a/app/assets/javascripts/application.js +++ b/app/assets/javascripts/application.js @@ -13,3 +13,5 @@ //= require rails-ujs //= require turbolinks //= require_tree . +//= require jquery +//= require bootstrap-sprockets \ No newline at end of file diff --git a/app/assets/stylesheets/application.css b/app/assets/stylesheets/application.scss similarity index 92% rename from app/assets/stylesheets/application.css rename to app/assets/stylesheets/application.scss index d05ea0f51..3befa3d45 100644 --- a/app/assets/stylesheets/application.css +++ b/app/assets/stylesheets/application.scss @@ -10,6 +10,6 @@ * files in this directory. Styles in this file should be added after the last require_* statement. * It is generally better to create a new file per style scope. * - *= require_tree . - *= require_self */ + @import 'bootstrap-sprockets'; + @import 'bootstrap'; diff --git a/app/assets/stylesheets/style.scss b/app/assets/stylesheets/style.scss new file mode 100644 index 000000000..25a20c212 --- /dev/null +++ b/app/assets/stylesheets/style.scss @@ -0,0 +1,55 @@ +html, body { + width: 100%; + height: 100%; +} + +body { + padding: 0px; + margin: 0px; + font-size: 14px; + line-height: 1.42857143; + color: #333; + background-color: #fff; +} + +* { + box-sizing: border-box; +} + +h1, h2, h3, h4, h5, h6, p { + margin: 0px; + padding: 0px; +} + +ul, ol { + list-style: none; + margin: 0px; + padding: 0px; +} + +a, a:hover, a:focus { + margin: 0px; + padding: 0px; + text-decoration: none; +} + +img { + width: 100%; + height: auto; + display: block; +} + +.container { + width: 85%; + max-width: 1170px; + margin: 0 auto; + position: relative; +} + +.col-left{ + float: left +} + +.col-right{ + float: right +} \ No newline at end of file diff --git a/app/controllers/admin/base_controller.rb b/app/controllers/admin/base_controller.rb index 4a89583f5..3760aed32 100644 --- a/app/controllers/admin/base_controller.rb +++ b/app/controllers/admin/base_controller.rb @@ -1,3 +1,12 @@ class Admin::BaseController < ApplicationController + before_action :authenticate_admin + private + def authenticate_admin + unless current_user.admin? + flash[:alert] = "Not allow" + redirect_to root_path + end + + end end diff --git a/app/controllers/admin/tweets_controller.rb b/app/controllers/admin/tweets_controller.rb index 24a57566c..be0a9a701 100644 --- a/app/controllers/admin/tweets_controller.rb +++ b/app/controllers/admin/tweets_controller.rb @@ -1,7 +1,20 @@ class Admin::TweetsController < Admin::BaseController def index + @tweets = Tweet.all.order(created_at: :desc) end def destroy + + @tweet = Tweet.find(params[:id]) + + @tweet.destroy + redirect_to admin_root_path + flash[:alert] = "tweet has destroyed" + + end + private + def tweet_params + params.require(:tweet).permit(:description) end + end diff --git a/app/controllers/admin/users_controller.rb b/app/controllers/admin/users_controller.rb index 3ba9f0a36..bf6188610 100644 --- a/app/controllers/admin/users_controller.rb +++ b/app/controllers/admin/users_controller.rb @@ -1,4 +1,8 @@ class Admin::UsersController < Admin::BaseController + def index + @users = User.all.order(tweets_count: :desc) end + + end diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb index 0da627f1a..0464e2c3b 100644 --- a/app/controllers/application_controller.rb +++ b/app/controllers/application_controller.rb @@ -1,7 +1,16 @@ class ApplicationController < ActionController::Base protect_from_forgery with: :exception + before_action :configure_permitted_parameters, if: :devise_controller? + before_action :authenticate_user! # 請參考 Devise 文件自訂表單後通過 Strong Parameters 的方法 # https://github.com/plataformatec/devise#strong-parameters # 注意有 sign_up 和 account_update 兩種參數要處理 + + protected + + def configure_permitted_parameters + devise_parameter_sanitizer.permit(:sign_up, keys: [:name]) + devise_parameter_sanitizer.permit(:account_update, keys: [:name]) + end end diff --git a/app/controllers/followships_controller.rb b/app/controllers/followships_controller.rb index 05f01b552..fdd001805 100644 --- a/app/controllers/followships_controller.rb +++ b/app/controllers/followships_controller.rb @@ -1,7 +1,25 @@ class FollowshipsController < ApplicationController + def create + @user = User.find(params[:following_id]) + if @user == current_user + flash[:alert] = "can't follow yourself" + else + @followship = current_user.followships.build(following_id: params[:following_id]) + if @followship.save + flash[:notice] = "followed sucessfully" + redirect_back(fallback_location: root_path) + else + flash[:alert] = @followship.errors.full_messages.to_sentence + redirect_back(fallback_location: root_path) + end + end end def destroy + @followship = current_user.followships.where(following_id: params[:id]).first + @followship.destroy + flash[:alert] = "Followship destroyed" + redirect_back(fallback_location: root_path) end end diff --git a/app/controllers/replies_controller.rb b/app/controllers/replies_controller.rb index a9b6a315b..ffe88e688 100644 --- a/app/controllers/replies_controller.rb +++ b/app/controllers/replies_controller.rb @@ -1,9 +1,32 @@ class RepliesController < ApplicationController def index + + @tweet = Tweet.find(params[:tweet_id]) + @replies = @tweet.replies + @reply = Reply.new + @user = @tweet.user + end def create + @tweet = Tweet.find(params[:tweet_id]) + @reply = @tweet.replies.build(reply_params) + @reply.user = current_user + + + if @reply.save + flash[:notice] = "reply successfully" + redirect_to tweet_replies_path + else + flash[:alert] = "fail to reply the tweet" + render :index + end + end + + private + def reply_params + params.require(:reply).permit(:comment, :user_id, :tweet_id) end end diff --git a/app/controllers/tweets_controller.rb b/app/controllers/tweets_controller.rb index ad14115c1..f159ce392 100644 --- a/app/controllers/tweets_controller.rb +++ b/app/controllers/tweets_controller.rb @@ -1,16 +1,41 @@ class TweetsController < ApplicationController def index - @users # 基於測試規格,必須講定變數名稱,請用此變數中存放關注人數 Top 10 的使用者資料 + @tweets = Tweet.all.order(created_at: :desc) + @tweet = Tweet.new + @users = User.all.order(followers_count: :desc).limit(10) # 基於測試規格,必須講定變數名稱,請用此變數中存放關注人數 Top 10 的使用者資料 end def create + @user = current_user + @tweet = @user.tweets.build(tweet_params) + + if @tweet.save + flash[:notice] = 'create a tweet successfully' + redirect_to tweets_path + else + flash[:alert] = 'failed to create a tweet' + render :index + end + end def like + @tweet = Tweet.find(params[:id]) + @tweet.likes.create!(user: current_user) + redirect_to tweets_path end def unlike + @tweet = Tweet.find(params[:id]) + likes = Like.where(tweet: @tweet, user: current_user) + likes.destroy_all + redirect_to tweets_path end + private + + def tweet_params + params.require(:tweet).permit(:description, :user_id) + end end diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb index 750e3c6b5..791ea2a97 100644 --- a/app/controllers/users_controller.rb +++ b/app/controllers/users_controller.rb @@ -1,24 +1,54 @@ class UsersController < ApplicationController def tweets + @user = User.find(params[:id]) + @tweet = Tweet.new + @tweets = @user.tweets.all end + + def edit + @user = User.find(params[:id]) + + unless @user == current_user + redirect_to tweets_user_path(@user) + end + end def update + @user = User.find(params[:id]) + @user.update(user_params) + + redirect_to tweets_user_path(@user) end def followings - @followings # 基於測試規格,必須講定變數名稱 + @user = User.find(params[:id]) + @followings = @user.followings.page(params[:page]).per(6).order('followships.created_at DESC') # 基於測試規格,必須講定變數名稱 end def followers - @followers # 基於測試規格,必須講定變數名稱 + @user = User.find(params[:id]) + @followers = @user.followers.page(params[:page]).per(6).order('followships.created_at DESC') # 基於測試規格,必須講定變數名稱 end def likes - @likes # 基於測試規格,必須講定變數名稱 + @user = User.find(params[:id]) + @likes = @user.liked_tweets.page(params[:page]).per(6).order('likes.created_at DESC') # 基於測試規格,必須講定變數名稱 + @tweets = @user.tweets + + + end + + def show + end + private + + def user_params + params.require(:user).permit(:name, :avatar, :introduction) + end end diff --git a/app/models/followship.rb b/app/models/followship.rb index 1aed01396..a7b080e92 100644 --- a/app/models/followship.rb +++ b/app/models/followship.rb @@ -1,4 +1,6 @@ class Followship < ApplicationRecord validates :following_id, uniqueness: { scope: :user_id } + belongs_to :user + belongs_to :following, class_name: "User", counter_cache: :followers_count end diff --git a/app/models/like.rb b/app/models/like.rb index d99b93a32..42d487577 100644 --- a/app/models/like.rb +++ b/app/models/like.rb @@ -1,2 +1,4 @@ class Like < ApplicationRecord + belongs_to :user + belongs_to :tweet, counter_cache: true end diff --git a/app/models/reply.rb b/app/models/reply.rb index bae6f9463..c7f16d932 100644 --- a/app/models/reply.rb +++ b/app/models/reply.rb @@ -1,2 +1,5 @@ class Reply < ApplicationRecord + validates_presence_of :comment + belongs_to :user + belongs_to :tweet, counter_cache: true end diff --git a/app/models/tweet.rb b/app/models/tweet.rb index 6715fada2..214364d44 100644 --- a/app/models/tweet.rb +++ b/app/models/tweet.rb @@ -1,4 +1,16 @@ class Tweet < ApplicationRecord + validates_presence_of :description validates_length_of :description, maximum: 140 + belongs_to :user, counter_cache: true + + has_many :replies, dependent: :destroy + + + has_many :likes, dependent: :destroy + has_many :liked_users, through: :likes, source: :user + + def is_liked?(user) + self.liked_users.include?(user) + end end diff --git a/app/models/user.rb b/app/models/user.rb index 6b05b8c21..72460a801 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -9,6 +9,27 @@ class User < ApplicationRecord # 需要 app/views/devise 裡找到樣板,加上 name 屬性 # 並參考 Devise 文件自訂表單後通過 Strong Parameters 的方法 validates_presence_of :name + validates_uniqueness_of :name # 加上驗證 name 不能重覆 (關鍵字提示: uniqueness) + has_many :tweets, dependent: :destroy + + has_many :replies, dependent: :restrict_with_error + + has_many :followships, dependent: :destroy + has_many :followings, through: :followships + + has_many :inverse_followships, class_name: "Followship",foreign_key: "following_id" + has_many :followers, through: :inverse_followships, source: :user + + has_many :likes, dependent: :destroy + has_many :liked_tweets, through: :likes, source: :tweet + + def admin? + self.role == "admin" + end + + def following?(user) + self.followings.include?(user) + end end diff --git a/app/views/admin/tweets/index.html.erb b/app/views/admin/tweets/index.html.erb new file mode 100644 index 000000000..28305d38e --- /dev/null +++ b/app/views/admin/tweets/index.html.erb @@ -0,0 +1,31 @@ + +
<%= simple_format(tweet.description)%>
+ <% tweet.replies.each do |reply| %> + <%= reply.comment %> + <% end %> +<%= link_to 'Reply', tweet_replies_path(tweet.id)%>
+<%= link_to 'Delete', admin_tweet_path(tweet), method: :delete, data:{confirm: "Are you sure?"} %>
+<%= link_to 'Tweet', tweets_user_path(user) %> <%= user.tweets.count %>
+<%= link_to 'Following',followings_user_path(user) %> <%= user.followings.count %>
+<%= link_to 'Follower',followers_user_path(user) %> <%= user.followers.count %>
+<%= link_to 'Like',likes_user_path(user) %> <%= user.likes.count %>
+<%= notice %>
<%= alert %>
<%= yield %> diff --git a/app/views/replies/index.html.erb b/app/views/replies/index.html.erb new file mode 100644 index 000000000..fbaa11ccf --- /dev/null +++ b/app/views/replies/index.html.erb @@ -0,0 +1,61 @@ +@<%= link_to @user.name, tweets_user_path(@tweet) %>
+<%= @user.introduction %>
+ +<%= simple_format(@tweet.description)%>
+ <%= link_to tweet_replies_path(@tweet) do %> + Reply (<%= @tweet.replies.count %>) + <% end %> + + <%= render partial: "shared/like", locals:{tweet: @tweet} %> +@<%= link_to reply.user.name, tweets_user_path(reply.user) %>, <%= reply.created_at.strftime("%Y-%m-%d, %H:%M") %>
+<%= reply.comment %>
+<%= link_to 'Tweets',tweets_user_path(user) %> <%= @user.tweets.count %>
+<%= link_to 'Following',followings_user_path(user) %> <%= @user.followings.count %>
+<%= link_to 'Follower', followers_user_path(user) %> <%= @user.followers.count %>
+<%= link_to 'Like', likes_user_path(user) %> <%= @user.likes.count %>
\ No newline at end of file diff --git a/app/views/shared/_follow.html.erb b/app/views/shared/_follow.html.erb new file mode 100644 index 000000000..8f6d4b9ad --- /dev/null +++ b/app/views/shared/_follow.html.erb @@ -0,0 +1,7 @@ +<% if user != current_user %> + <% if current_user.following?(user) %> + <%= link_to "Unfollow", followship_path(user), method: :delete, class: "btn btn-primary" %> + <% else %> + <%= link_to "Follow", followships_path(following_id: user), method: :post, class: "btn btn-primary" %> + <% end %> +<% end %> \ No newline at end of file diff --git a/app/views/shared/_like.html.erb b/app/views/shared/_like.html.erb new file mode 100644 index 000000000..da8c1bb05 --- /dev/null +++ b/app/views/shared/_like.html.erb @@ -0,0 +1,10 @@ + +<% if tweet.is_liked?(current_user) %> + <%= link_to unlike_tweet_path(tweet),method: :post do %> + Unlike(<%= tweet.likes.count %>) + <% end %> +<% else %> + <%= link_to like_tweet_path(tweet), method: :post do %> + Like (<%= tweet.likes.count %>) + <% end %> +<% end %> diff --git a/app/views/tweets/index.html.erb b/app/views/tweets/index.html.erb new file mode 100644 index 000000000..426b344b6 --- /dev/null +++ b/app/views/tweets/index.html.erb @@ -0,0 +1,65 @@ +<%= simple_format(tweet.description)%>
+ <%= link_to tweet_replies_path(tweet) do%> + Reply (<%= tweet.replies.count %>) + <% end %> + + <%= render partial: "shared/like", locals:{tweet: tweet} %> +@<%= link_to user.name, tweets_user_path(user) %>
+<%= user.introduction %>
+@<%= link_to @user.name, tweets_user_path(@user) %>
+<%= @user.introduction %>
+ +@<%= link_to @user.name, tweets_user_path(@user) %>
+<%= @user.introduction %>
+ +@<%= link_to @user.name, tweets_user_path(@user) %>
+<%= @user.introduction %>
+ +@<%= link_to tweet.user.name, tweets_user_path(tweet.user) %>, <%= tweet.created_at.strftime("%Y-%m_%d, %H:%M") %>
+<%= tweet.user.introduction %>
+@<%= link_to @user.name, tweets_user_path(@user) %> +
<%= @user.introduction %>
+ +@<%= link_to tweet.user.name, tweets_user_path(tweet.user) %>, <%= tweet.created_at.strftime("%Y-%m-%d, %H:%M") %>
+<%= tweet.description %>
+