change routes to properly reflect page names (activity/stream).

This commit is contained in:
danielgrippi 2012-02-14 11:16:27 -08:00
parent e24873353b
commit f3f2b8ca44
37 changed files with 64 additions and 62 deletions

View file

@ -88,7 +88,7 @@ class ApplicationController < ActionController::Base
def redirect_unless_admin
unless current_user.admin?
redirect_to explore_url, :notice => 'you need to be an admin to do that'
redirect_to stream_url, :notice => 'you need to be an admin to do that'
return
end
end
@ -116,7 +116,7 @@ class ApplicationController < ActionController::Base
end
def after_sign_in_path_for(resource)
stored_location_for(:user) || (current_user.getting_started? ? getting_started_path : explore_path)
stored_location_for(:user) || (current_user.getting_started? ? getting_started_path : stream_path)
end
def max_time

View file

@ -6,7 +6,7 @@ class HomeController < ApplicationController
def show
if current_user
redirect_to explore_path if current_user
redirect_to stream_path if current_user
elsif is_mobile_device?
redirect_to user_session_path
else

View file

@ -52,7 +52,7 @@ class NotificationsController < ApplicationController
def read_all
Notification.where(:recipient_id => current_user.id).update_all(:unread => false)
respond_to do |format|
format.html { redirect_to explore_path }
format.html { redirect_to stream_path }
format.xml { render :xml => {}.to_xml }
format.json { render :json => {}.to_json }
end

View file

@ -50,7 +50,7 @@ class PostsController < ApplicationController
respond_to do |format|
format.js {render 'destroy'}
format.json { render :nothing => true, :status => 204 }
format.all {redirect_to explore_path}
format.all {redirect_to stream_path}
end
else
Rails.logger.info "event=post_destroy status=failure user=#{current_user.diaspora_handle} reason='User does not own post'"

View file

@ -66,7 +66,7 @@ class StatusMessagesController < ApplicationController
respond_to do |format|
format.html { redirect_to :back}
format.mobile{ redirect_to explore_path}
format.mobile{ redirect_to stream_path}
format.json{ render :json => @status_message.as_api_response(:backbone), :status => 201 }
end
else

View file

@ -8,7 +8,7 @@ require File.join(Rails.root, "lib", "stream", "comments")
require File.join(Rails.root, "lib", "stream", "likes")
require File.join(Rails.root, "lib", "stream", "mention")
require File.join(Rails.root, "lib", "stream", "followed_tag")
require File.join(Rails.root, "lib", "stream", "participate")
require File.join(Rails.root, "lib", "stream", "activity")
class StreamsController < ApplicationController
@ -31,8 +31,8 @@ class StreamsController < ApplicationController
stream_responder(Stream::Public)
end
def participate
stream_responder(Stream::Participate)
def activity
stream_responder(Stream::Activity)
end
def multi

View file

@ -63,6 +63,6 @@ class TagFollowingsController < ApplicationController
@tag_following = current_user.tag_followings.create(:tag_id => @tag.id)
end
end
redirect_to explore_path
redirect_to stream_path
end
end

View file

@ -87,7 +87,7 @@ class UsersController < ApplicationController
if params[:user] && params[:user][:current_password] && current_user.valid_password?(params[:user][:current_password])
current_user.close_account!
sign_out current_user
redirect_to(explore_path, :notice => I18n.t('users.destroy.success'))
redirect_to(stream_path, :notice => I18n.t('users.destroy.success'))
else
if params[:user].present? && params[:user][:current_password].present?
flash[:error] = t 'users.destroy.wrong_password'
@ -111,7 +111,7 @@ class UsersController < ApplicationController
format.any { redirect_to person_path(user.person) }
end
else
redirect_to explore_path, :error => I18n.t('users.public.does_not_exist', :username => params[:username])
redirect_to stream_path, :error => I18n.t('users.public.does_not_exist', :username => params[:username])
end
end
@ -127,14 +127,14 @@ class UsersController < ApplicationController
def logged_out
@page = :logged_out
if user_signed_in?
redirect_to explore_path
redirect_to stream_path
end
end
def getting_started_completed
user = current_user
user.update_attributes(:getting_started => false)
redirect_to explore_path
redirect_to stream_path
end
def export

View file

@ -13,7 +13,7 @@ module StreamHelper
elsif controller.instance_of?(PostsController)
public_stream_path(:max_time => time_for_scroll(@stream))
elsif controller.instance_of?(StreamsController)
explore_path(:max_time => time_for_scroll(@stream))
stream_path(:max_time => time_for_scroll(@stream))
else
raise 'in order to use pagination for this new controller, update next_page_path in stream helper'
end

View file

@ -49,7 +49,7 @@
.navbar.navbar-fixed-top
.navbar-inner
.container{:style => "position: relative;"}
= link_to(image_tag('header-logo2x.png', :height => 40, :width => 40, :id => 'header_title'), explore_path)
= link_to(image_tag('header-logo2x.png', :height => 40, :width => 40, :id => 'header_title'), stream_path)
- if user_signed_in?
.right

View file

@ -31,7 +31,7 @@
.section
%ul.left_nav
%li
= link_to t("streams.participate.title"), participate_path, :class => 'home_selector', :rel => 'backbone'
= link_to t("streams.activity.title"), activity_stream_path, :class => 'home_selector', :rel => 'backbone'
%ul.left_nav.sub
%li
@ -41,7 +41,7 @@
%ul.left_nav
%li
= link_to t("streams.multi.title"), explore_path, :class => 'home_selector', :rel => 'backbone'
= link_to t("streams.multi.title"), stream_path, :class => 'home_selector', :rel => 'backbone'
= render 'aspects/aspect_listings', :stream => @stream

View file

@ -22,7 +22,7 @@
%h4
= t('.simply_visit')
%strong
= link_to AppConfig[:pod_url], explore_path
= link_to AppConfig[:pod_url], stream_path
= t('.on_your_mobile_device')
%p.dull

View file

@ -927,14 +927,13 @@ en:
multi:
title: "Stream"
contacts_title: "People in your Explore Stream"
contacts_title: "People in your Stream"
aspects:
title: "My Aspects"
participate:
activity:
title: "My Activity"
users:
logged_out:
signed_out: "You've signed out of Diaspora*"

View file

@ -22,8 +22,11 @@ Diaspora::Application.routes.draw do
end
# Streams
get "participate" => "streams#participate", :as => "participate"
get "explore" => "streams#multi", :as => "explore"
get "participate" => "streams#activity", :as => "activity_stream" # legacy
get "explore" => "streams#multi", :as => "stream" # legacy
get "activity" => "streams#activity", :as => "activity_stream"
get "stream" => "streams#multi", :as => "stream"
get "public" => "streams#public", :as => "public_stream"
get "followed_tags" => "streams#followed_tags", :as => "followed_tags_stream"
get "mentions" => "streams#mentioned", :as => "mentioned_stream"

View file

@ -16,7 +16,7 @@ Feature: invitation acceptance
And I preemptively confirm the alert
And I follow "awesome_button"
Then I should be on the explore page
Then I should be on the stream page
Scenario: accept invitation from user
Given I have been invited by a user
@ -34,7 +34,7 @@ Feature: invitation acceptance
And I preemptively confirm the alert
And I follow "awesome_button"
Then I should be on the explore page
Then I should be on the stream page
Scenario: sends an invitation
Given a user with email "bob@bob.bob"

View file

@ -1,5 +1,5 @@
@javascript
Feature: The participate stream
Feature: The activity stream
Scenario: Sorting
Given a user with username "bob"
When I sign in as "bob@bob.bob"
@ -8,7 +8,7 @@ Feature: The participate stream
And I post "B- barack obama is your new bicycle"
And I post "C- barack obama is a square"
When I go to the participate page
When I go to the activity stream page
Then "C- barack obama is a square" should be post 1
And "B- barack obama is your new bicycle" should be post 2
And "A- I like turtles" should be post 3
@ -20,7 +20,7 @@ Feature: The participate stream
And I like the post "B- barack obama is your new bicycle"
And I wait for 1 second
When I go to the participate page
When I go to the activity stream page
Then "B- barack obama is your new bicycle" should be post 1
And "C- barack obama is a square" should be post 2
And "A- I like turtles" should be post 3

View file

@ -11,7 +11,7 @@ Feature: Change password
Then I should see "Password changed"
Then I should be on the new user session page
When I sign in with password "newsecret"
Then I should be on the explore page
Then I should be on the stream page
Scenario: Reset my password
Given a user with email "forgetful@users.net"

View file

@ -6,7 +6,7 @@ Feature: user authentication
And I fill in "Username" with "ohai"
And I fill in "Password" with "secret"
And I press "Sign in"
Then I should be on the explore page
Then I should be on the stream page
@javascript
Scenario: user logs out
@ -19,5 +19,5 @@ Feature: user authentication
Scenario: user uses token auth
Given a user with username "ohai" and password "secret"
When I post a photo with a token
And I go to the explore page
And I go to the stream page
Then I should be on the new user session page

View file

@ -19,18 +19,18 @@ Feature: new user registration
And I preemptively confirm the alert
And I follow "awesome_button"
Then I should be on the explore page
Then I should be on the stream page
And I should not see "awesome_button"
Scenario: new user skips the setup wizard
When I preemptively confirm the alert
And I follow "awesome_button"
Then I should be on the explore page
Then I should be on the stream page
Scenario: closing a popover clears getting started
When I preemptively confirm the alert
And I follow "awesome_button"
Then I should be on the explore page
Then I should be on the stream page
And I have turned off jQuery effects
And I wait for the popovers to appear
And I click close on all the popovers

View file

@ -2,7 +2,7 @@ module NavigationHelpers
def path_to(page_name)
case page_name
when /^the home(?: )?page$/
explore_path
stream_path
when /^step (\d)$/
if $1.to_i == 1
getting_started_path

View file

@ -1,6 +1,6 @@
class Stream::Participate < Stream::Base
class Stream::Activity < Stream::Base
def link(opts={})
Rails.application.routes.url_helpers.participate_path(opts)
Rails.application.routes.url_helpers.activity_streams_path(opts)
end
def order
@ -8,7 +8,7 @@ class Stream::Participate < Stream::Base
end
def title
I18n.translate("streams.participate.title")
I18n.translate("streams.activity.title")
end
# @return [ActiveRecord::Association<Post>] AR association of posts

View file

@ -2,7 +2,7 @@ class Stream::Multi < Stream::Base
# @return [String] URL
def link(opts)
Rails.application.routes.url_helpers.explore_path(opts)
Rails.application.routes.url_helpers.stream_path(opts)
end
# @return [String]

View file

@ -58,7 +58,7 @@ app.models.Stream = Backbone.Collection.extend({
},
sortOrder : function() {
return this.basePath().match(/participate/) ? "interactedAt" : "createdAt"
return this.basePath().match(/activity/) ? "interactedAt" : "createdAt"
},
add : function(models){

View file

@ -1,7 +1,7 @@
app.Router = Backbone.Router.extend({
routes: {
"participate": "stream",
"explore": "stream",
"activity": "stream",
"stream": "stream",
"aspects:query": "stream",
"commented": "stream",
"liked": "stream",

View file

@ -1,18 +1,18 @@
<div class="container" style="position:relative;">
<a href="/explore">
<a href="/stream">
<img alt="Logo_small" class="diaspora_header_logo" height="38px" width="65px" src="{{imageUrl "/images/header-logo.png"}}" />
</a>
<span class="header-nav">
<span>
<a href="/explore">
<a href="/stream">
{{t "my_stream"}}
</a>
</span>
<span>
<a href="/participate">
<a href="/activity">
{{t "my_activity"}}
</a>
</span>

View file

@ -14,7 +14,7 @@ describe AdminsController do
context 'admin not signed in' do
it 'is behind redirect_unless_admin' do
get :user_search
response.should redirect_to explore_path
response.should redirect_to stream_path
end
end
@ -64,7 +64,7 @@ describe AdminsController do
context 'admin not signed in' do
it 'is behind redirect_unless_admin' do
get :admin_inviter
response.should redirect_to explore_path
response.should redirect_to stream_path
end
end

View file

@ -14,7 +14,7 @@ describe HomeController do
it 'redirects to multis index if user is logged in' do
sign_in alice
get :show, :home => true
response.should redirect_to(explore_path)
response.should redirect_to(stream_path)
end
end
end

View file

@ -10,7 +10,7 @@ describe StreamsController do
sign_in :user, alice
end
it 'generates the explore_json fixture', :fixture => true do
it 'generates the stream_json fixture', :fixture => true do
posts = []
time = Time.now
@ -53,7 +53,7 @@ TXT
get :multi, :format => :json
response.should be_success
save_fixture(response.body, "explore_json")
save_fixture(response.body, "stream_json")
Timecop.return
end

View file

@ -59,7 +59,7 @@ describe NotificationsController do
it "should redirect to the stream in the html version" do
Factory(:notification, :recipient => @user)
get :read_all, :format => :html
response.should redirect_to(explore_path)
response.should redirect_to(stream_path)
end
it "should return a dummy value in the json version" do
Factory(:notification, :recipient => @user)

View file

@ -67,7 +67,7 @@ describe RegistrationsController do
it "redirects to the home path" do
get :create, @valid_params
response.should be_redirect
response.location.should match /^#{explore_url}\??$/
response.location.should match /^#{stream_url}\??$/
end
end

View file

@ -27,14 +27,14 @@ describe SessionsController do
it "redirects to /stream for a non-mobile user" do
post :create, {"user" => {"remember_me" => "0", "username" => @user.username, "password" => "evankorth"}}
response.should be_redirect
response.location.should match /^#{explore_url}\??$/
response.location.should match /^#{stream_url}\??$/
end
it "redirects to /stream for a mobile user" do
@request.env['HTTP_USER_AGENT'] = 'Mozilla/5.0 (iPhone; U; CPU iPhone OS 4_1 like Mac OS X; en-us) AppleWebKit/532.9 (KHTML, like Gecko) Version/4.0.5 Mobile/8B117 Safari/6531.22.7'
post :create, {"user" => {"remember_me" => "0", "username" => @user.username, "password" => "evankorth"}}
response.should be_redirect
response.location.should match /^#{explore_url}\??$/
response.location.should match /^#{stream_url}\??$/
end
it 'queues up an update job' do

View file

@ -54,7 +54,7 @@ describe StreamsController do
:liked => Stream::Likes,
:mentioned => Stream::Mention,
:followed_tags => Stream::FollowedTag,
:participate => Stream::Participate
:activity => Stream::Activity
}
streams.each do |stream_path, stream_class|

View file

@ -9,7 +9,7 @@ describe("app.views.Feedback", function(){
'limited' : "Limted"
}})
var posts = $.parseJSON(spec.readFixture("explore_json"))["posts"];
var posts = $.parseJSON(spec.readFixture("stream_json"))["posts"];
this.post = new app.models.Post(posts[0]);
this.view = new app.views.Feedback({model: this.post});

View file

@ -9,7 +9,7 @@ describe("app.views.LikesInfo", function(){
}
})
var posts = $.parseJSON(spec.readFixture("explore_json"))["posts"];
var posts = $.parseJSON(spec.readFixture("stream_json"))["posts"];
this.post = new app.models.Post(posts[0]); // post with a like
this.view = new app.views.LikesInfo({model: this.post});
});

View file

@ -11,7 +11,7 @@ describe("app.views.Post", function(){
}
}})
var posts = $.parseJSON(spec.readFixture("explore_json"))["posts"];
var posts = $.parseJSON(spec.readFixture("stream_json"))["posts"];
this.collection = new app.collections.Posts(posts);
this.statusMessage = this.collection.models[0];

View file

@ -2,7 +2,7 @@ describe("app.views.Stream", function() {
beforeEach(function() {
loginAs({name: "alice", avatar : {small : "http://avatar.com/photo.jpg"}});
this.posts = $.parseJSON(spec.readFixture("explore_json"))["posts"];
this.posts = $.parseJSON(spec.readFixture("stream_json"))["posts"];
this.stream = new app.models.Stream();
this.stream.add(this.posts);

View file

@ -1,9 +1,9 @@
require 'spec_helper'
require File.join(Rails.root, 'spec', 'shared_behaviors', 'stream')
describe Stream::Participate do
describe Stream::Activity do
before do
@stream = Stream::Participate.new(alice, :max_time => Time.now, :order => 'updated_at')
@stream = Stream::Activity.new(alice, :max_time => Time.now, :order => 'updated_at')
end
describe 'shared behaviors' do