Skip to content
This repository has been archived by the owner on Mar 29, 2019. It is now read-only.

Begin Rails 3 work #256

Closed
wants to merge 76 commits into from
Closed

Conversation

thegcat
Copy link
Member

@thegcat thegcat commented Jul 8, 2013

This combines some Rails 3 work from @meineerde and me, this is a beginning but still needs a lot of work and is far from stable.

Current Ruby and Rails status: The current targets are Ruby 1.9.3 (1.8 is EOL) and Rails 3.2-latest. Ruby support can be expanded to later Ruby versions once it's stable on 1.9.3.

meineerde and others added 30 commits July 8, 2013 16:46
awesome_nested_set would have needed to be updated for rails 3.1,
unvendoring it makes updates easier. The patches to the vendored
awesome_nested_set will have to be reviewed and adapted.
…df plugin (#4796)

ActionView::TemplateHandlers is deprecated on Rails3.2
and Redmine cannot boot on Rails3.2.
Redmine does not use this rfpdf plugin feature.

git-svn-id: svn://rubyforge.org/var/svn/redmine/trunk@8702 e93f8b46-1217-0410-a6f0-8f06a7374b81
Original code by Jean-Philippe Lang
redmine:b590ff105e2e69ccd5b06c2783860e78bc1b392b
…dlers.

git-svn-id: svn://rubyforge.org/var/svn/redmine/trunk@9604 e93f8b46-1217-0410-a6f0-8f06a7374b81
The bundled open_id_authentication plugin has some Rails 3
incompatibilities.
As partials might also be used in js formatted responses, we just
rename them to .erb. Normal templates are renamed to .html.erb.
This follows the new Rails convention to play well with tilt.

for i in `find app/views/**/_*.rhtml`; do \
  git mv $i `echo $i | sed s/\\.rhtml$/.erb/`; \
done

for i in `find app/views/**/*.rhtml`; do \
  git mv $i `echo $i | sed s/\\.rhtml$/.html.erb/`; \
done
This is required to work around a circular dependency issue during class loading.
See the comment in application.rb for details.
Rails 3 uses a secret token in various places. The former cookie
secret has thus been re-purposed to a generic secret token which
is generated with the new task.

The former task also configured the cookie session store. this is
now done in application.rb and can be overridden in the
additional_environment.rb
@marutosi
Copy link
Contributor

marutosi commented Jul 8, 2013

I disagree this pull request.
You should not push large changes to central repository.

You should use rebase as we did in Redmine.
So, you should use forked repository.

@marutosi
Copy link
Contributor

marutosi commented Jul 8, 2013

Why don't you cherry-pick 4d186c2 and 04f05de etc?
These revisions are not depend on Rails3.

@thegcat
Copy link
Member Author

thegcat commented Jul 9, 2013

@marutosi this is a rebase of the porting work on unstable and won't be merged as one big commit but as it is, I'm sorry but I'm not sure I understand the objection.

@thegcat
Copy link
Member Author

thegcat commented Jul 16, 2013

Commit 4dd3d40 added to master in dff8cf6.

@meineerde meineerde closed this Feb 2, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants