Release OpenProject 8.2.0

Author: oliverguenther

.gitignore

@@ -41,6 +41,9 @@ npm-debug.log*
 /tmp
 *.swp
 
+# Ignore Visual Studio Code files
+/.vscode
+
 # Ignore RubyMine files
 /.idea
 /frontend/.idea

.travis.yml

@@ -26,16 +26,6 @@
 # See doc/COPYRIGHT.rdoc for more details.
 #++
 
-
-###################################
-#
-# This file was generated by
-# openproject-devkit.
-#
-# Do not modify this file directly!
-#
-###################################
-
 language: ruby
 
 rvm:
@@ -189,40 +179,37 @@ jobs:
       script:
       - bash script/ci/setup.sh plugins:units mysql
       - bash script/ci/runner.sh plugins:units 1 1
+      if: head_branch !~ /^core\//
     - stage: test
       name: 'plugins:units (1/1) - postgres'
       script:
       - bash script/ci/setup.sh plugins:units postgres
       - bash script/ci/runner.sh plugins:units 1 1
+      if: head_branch !~ /^core\//
     - stage: test
       name: 'plugins:features (1/1) - mysql'
       script:
       - bash script/ci/setup.sh plugins:features mysql
       - bash script/ci/runner.sh plugins:features 1 1
+      if: head_branch !~ /^core\//
     - stage: test
       name: 'plugins:features (1/1) - postgres'
       script:
       - bash script/ci/setup.sh plugins:features postgres
       - bash script/ci/runner.sh plugins:features 1 1
+      if: head_branch !~ /^core\//
     - stage: test
       name: 'plugins:cucumber (1/1) - mysql'
       script:
       - bash script/ci/setup.sh plugins:cucumber mysql
       - bash script/ci/runner.sh plugins:cucumber 1 1
+      if: head_branch !~ /^core\//
     - stage: test
       name: 'plugins:cucumber (1/1) - postgres'
       script:
       - bash script/ci/setup.sh plugins:cucumber postgres
       - bash script/ci/runner.sh plugins:cucumber 1 1
-
-notifications:
-  email: false
-  slack:
-    on_success: change
-    on_failure: always
-    on_pull_requests: false
-    rooms:
-      - secure: DzTFQBORAgS6ncPWuP6j60rDyHq/Fw4STPPi8CyQCbpNnqJKtbznJJqZH5JQMHAHE7ak2awnGwK3x4Ls0SmSUOxxKn1r3LGNP6INdvHbf7YG7Vk247+uau4xOx/JDS0ueb9VwYyHvD8TM9Jh1BYaPvCmvTh9WBWCnlWFteAyGT8cpAtMtVRK/GFS4QWukG/uBo/lWI/Lw4DU5jPZonj/mW9XconT3qCqHsRZ8LgIF/3zOzO2b6N9R2mbyHmOqO+X36XsFL5lrfH5A7Z9ZZ5UZNqzvhjf/Ft2NLLv9RVQMleFlGEavCcr5sCXQRrb6ffmKKJoVilopvmpuWaVRkVqx8Wl6qCjXUefeiHXCM6+HWFO9I+xvh59e/Dl4mPASCQOamOuNjAva2JgpxP1aifDvt2jdAjPagf2Ss5csRsR2pq5VR/eyDu8U03hZSzphDqSgCqxRJtBv50V7KByTFO6u3AefYw71EZYX6khKFCZZDQ+GAZLdZGvhZXzecejMJETTz+f8m8Nd+M+uidCnND+vDKoZDxI8lSDE7zYdJovpf2M2vUQbBUgKtKOfoY22YqG3O77/HhrVrS8hoJAfdE/wSGYktYQre90VWkekBiU57/PXqSU2hnRFGytWonfNi1M89Ib5F4Vsr/AuBvXx2//PkOpiZwJFC/LT9d394Ls3Mo=
+      if: head_branch !~ /^core\//
 
 addons:
   chrome: stable

Dockerfile

@@ -18,8 +18,7 @@ RUN apt-get update -qq && \
 	poppler-utils \
 	unrtf \
 	tesseract-ocr \
-	catdoc \
-	pandoc && \
+	catdoc && \
 	apt-get clean && rm -rf /var/lib/apt/lists/*
 
 # using /home/app since npm cache and other stuff will be put there when running npm install
@@ -32,8 +31,11 @@ WORKDIR $APP_PATH
 
 COPY Gemfile ./Gemfile
 COPY Gemfile.* ./
-COPY vendored-plugins ./vendored-plugins
-RUN bundle install --jobs 8 --retry 3 --with docker
+COPY modules ./modules
+# OpenProject::Version is required by module versions in gemspecs
+RUN mkdir -p lib/open_project
+COPY lib/open_project/version.rb ./lib/open_project/
+RUN bundle install --deployment --with="docker opf_plugins" --without="test development" --jobs=8 --retry=3
 
 # Then, npm install node modules
 COPY package.json /tmp/npm/package.json
@@ -48,9 +50,22 @@ RUN cp docker/Procfile .
 RUN cp packaging/conf/database.yml config/
 RUN sed -i "s|Rails.groups(:opf_plugins)|Rails.groups(:opf_plugins, :docker)|" config/application.rb
 
+# Ensure we can write in /tmp/op_uploaded_files (cf. #29112)
+RUN mkdir -p /tmp/op_uploaded_files/
+RUN chown -R $APP_USER:$APP_USER /tmp/op_uploaded_files/
+
+# Allow uploading avatars w/ postgres
+RUN chown -R $APP_USER:$APP_USER $APP_DATA
+
+# Re-use packager database.yml
+COPY packaging/conf/database.yml ./config/database.yml
+
 # Run the npm postinstall manually after it was copied
 RUN DATABASE_URL=sqlite3:///tmp/db.sqlite3 SECRET_TOKEN=foobar RAILS_ENV=production bundle exec rake assets:precompile
 
+# Include pandoc
+RUN DATABASE_URL=sqlite3:///tmp/db.sqlite3 RAILS_ENV=production bundle exec rails runner "puts ::OpenProject::TextFormatting::Formats::Markdown::PandocDownloader.check_or_download!"
+
 CMD ["./docker/web"]
 ENTRYPOINT ["./docker/entrypoint.sh"]
 VOLUME ["$APP_DATA"]

Gemfile

@@ -33,7 +33,7 @@ ruby '~> 2.5.1'
 gem 'actionpack-xml_parser', '~> 2.0.0'
 gem 'activemodel-serializers-xml', '~> 1.0.1'
 gem 'activerecord-session_store', '~> 1.1.0'
-gem 'rails', '~> 5.1.5'
+gem 'rails', '~> 5.1.6'
 gem 'responders', '~> 2.4'
 
 gem 'rubytree', git: 'https://github.com/dr0verride/RubyTree.git', ref: '06f53ee'
@@ -195,7 +195,7 @@ group :test do
 
   # Test prof provides factories from code
   # and other niceties
-  gem 'test-prof', '~> 0.4.0'
+  gem 'test-prof', '~> 0.7.3'
 
   gem 'cucumber', '~> 3.0.0'
   gem 'cucumber-rails', '~> 1.6.0', require: false
@@ -217,11 +217,11 @@ group :test do
   # brings back testing for 'assigns' and 'assert_template' extracted in rails 5
   gem 'rails-controller-testing', '~> 1.0.2'
 
-  gem 'capybara', '~> 3.0.0'
+  gem 'capybara', '~> 3.11.1'
   gem 'capybara-screenshot', '~> 1.0.17'
   gem 'capybara-select2', git: 'https://github.com/goodwill/capybara-select2', ref: '585192e'
-  gem 'chromedriver-helper', '~> 1.2.0'
-  gem 'selenium-webdriver', '~> 3.11'
+  gem 'chromedriver-helper', '~> 2.1.0'
+  gem 'selenium-webdriver', '~> 3.14'
 
   gem 'fuubar', '~> 2.3.1'
   gem 'timecop', '~> 0.9.0'
@@ -250,6 +250,13 @@ end
 
 group :development, :test do
   gem 'thin', '~> 1.7.2'
+  gem 'ruby-prof', require: false
+  gem 'puma', '~> 3.11.3'
+
+  # Tracing and profiling gems
+  gem 'rack-mini-profiler', require: false
+  gem 'flamegraph', require: false
+  gem 'stackprof', require: false
 
   gem 'pry-rails', '~> 0.3.6'
   gem 'pry-stack_explorer', '~> 0.4.9.2'
@@ -290,7 +297,7 @@ group :docker, optional: true do
 end
 
 # Load Gemfile.local, Gemfile.plugins, plugins', and custom Gemfiles
-gemfiles = Dir.glob File.expand_path('../{Gemfile.local,Gemfile.plugins,lib/plugins/*/Gemfile}',
+gemfiles = Dir.glob File.expand_path('../{Gemfile.plugins,Gemfile.modules,Gemfile.local,lib/plugins/*/Gemfile}',
                                      __FILE__)
 gemfiles << ENV['CUSTOM_PLUGIN_GEMFILE'] unless ENV['CUSTOM_PLUGIN_GEMFILE'].nil?
 gemfiles.each do |file|