1. 19 Jun, 2019 1 commit
  2. 23 May, 2019 1 commit
  3. 10 May, 2019 4 commits
    • dgt's avatar
      Update: Add some new Rails 5.2 config params · 044d5d53
      dgt authored
      044d5d53
    • dgt's avatar
      Get rid of some deprecation warnings · 17e68d94
      dgt authored
      17e68d94
    • dgt's avatar
      Some preparations for rails 5.1 · f0bd3b4d
      dgt authored
      update some gems, replace before_filter / after_filter by before_action / after_action
      and some more config changes (mostly commented out) and scripts generated by rails app:update task
      f0bd3b4d
    • dgt's avatar
      Preparation for Rails 5 · 7a912d4c
      dgt authored
      first make everything work locally with rails 5.0
      test do not pass on CI yet because the test task has to be rewritten
      
      - update dependecies
      - run rails app:update task: update config, generate intitializers etc.
      - the method hide_action is not available in rails anymore
      - ActionController::Parameters have to be converted to a hash
      - protect_from_forgery now defaults to prepend:false
      7a912d4c
  4. 18 Jun, 2018 1 commit
  5. 10 Jun, 2018 1 commit
  6. 16 Oct, 2017 1 commit
  7. 12 Sep, 2017 2 commits
  8. 01 Sep, 2016 14 commits
    • azul's avatar
      use hybrid Cookies serializer · e702284a
      azul authored
      Slowly migrate towards the json one.
      e702284a
    • azul's avatar
      upgrade: use the new config/secrets.yml · 470787d3
      azul authored
      Still reading secret_token from config/crabgrass/secret.txt if it is there.
      
      This way we can migrate old sessions to the new keybase.
      470787d3
    • azul's avatar
      upgrade: dump db as ruby not sql · 0c110f93
      azul authored
      Let's see if this fixes travis missing db/structure.sql
      0c110f93
    • azul's avatar
      upgrade: application.rb Gemfile without :assets now · 51acf593
      azul authored
      51acf593
    • azul's avatar
      upgrade: disable_implicit_join_references in AR · 4adf7c1d
      azul authored
      This option turns of the implicit joining of references mentioned in
      where queries. As the deprecation comment of the feature states it
      is inherently buggy and we don't seem to rely on it so far.
      
      So let's just turn it of for the future.
      4adf7c1d
    • azul's avatar
      upgrade: use namespace for notices · 4aea05e3
      azul authored
      4aea05e3
    • azul's avatar
      proper namespace for polls · 30cf1bb2
      azul authored
      30cf1bb2
    • azul's avatar
      TaskParticipation -> Task::Participation · 74e7c1c2
      azul authored
      cleans up circular dependency:
      RuntimeError: Circular dependency detected while autoloading constant Task::TaskParticipation
      74e7c1c2
    • azul's avatar
      fix discussion namespace issue · 65bc1cd6
      azul authored
      or rather lack of namespace issue.
      We'll get rid of discussions and move posts to messages and comments
      so no point in trying to create a properly namespaced version of them.
      65bc1cd6
    • azul's avatar
      upgrade: drop outdated config settings (rails 4.0) · 6937f342
      azul authored
      6937f342
    • azul's avatar
      move the WikiExtensions into Wiki namespace · 8933b86c
      azul authored
      8933b86c
    • azul's avatar
      move remaining associations into user namespace · 3aa324a7
      azul authored
      3aa324a7
    • azul's avatar
      namespace: asset for all the asset things · 3c9d9525
      azul authored
      Also separate Asset::Storage::Path from Asset::Storage
      3c9d9525
    • azul's avatar
      namespace Activity and remove from autoload_path · 4955ee0a
      azul authored
      We used to have app/models/activity in the autoload path and had classes
      like FriendActivity < Activity
      
      This conflicts with how rails4 handles class lookup. Plus it clutters the
      autoload path and makes it harder to find classes.
      
      Using proper namespaces instead now and I'm about to apply this to all the
      different namespace avoiding things like ...Extension. We can use classes
      as namespaces as long as we:
      * make sure to define things inside the namespace in one line or in a
         class instead of a module block:
         ```ruby
           class Activity::Friend < Activity
           # or
           class Activity
             class Friend < ::Activity
         ```
       * have names inside the namespaces that do not conflict with global names
      
      The second one is important because if PrivatePost is already loaded and
      Activity::PrivatePost is referenced somewhere it will be found as an
      constant inside the Object namespace of Activity (since Activity is a class).
      So instead of loading Activity::PrivatePost we'll see a warning that global
      constant PrivatePost was referenced by Activity::PrivatePost.
      
      So how do we prevent this?
      * do not clutter the global namespace
      * use more specific names in other namespaces
        -> in this case i am renaming PrivatePostActivity to Activity::MessageSent
        and not Activity::PrivatePost
      4955ee0a
  9. 28 Oct, 2015 1 commit
  10. 21 Jul, 2015 1 commit
  11. 25 Jun, 2015 1 commit
    • azul's avatar
      fix castle gates reloading in development · 043c351c
      azul authored
      Here's what was going on...
      
      When loading a page after changing a file in development the server
      would crash indicating that config/permissions.rb was loaded twice.
      Behind the scenes rails was trying to load the constant Group inside
      the ContextPagesController. So it first looked up
        ContextPagesController::Group
      and then Group itself. Both lookups fired the after_reload callbacks.
      This way the callbacks for Group were run twice leading to errors because
      duplicate gates were defined.
      
      What are we doing now?...
      
      Now we load config/permissions.rb inside a config.to_prepare block.
      Here's what the documentation says about that hook:
      ```
        to_prepare: Run after the initializers are run for all Railties
        (including the application itself), but before eager loading and
        the middleware stack is built.
        More importantly, will run upon every request in development, but
        only once (during boot-up) in production and test.
      ```
      
      We also use require_dependency instead of plain require. This way the file
      will actually be reloaded when models are reloaded.
      043c351c
  12. 23 Jun, 2015 3 commits
    • azul's avatar
      namespace all the tracking things · 4668ea6b
      azul authored
      Tracking now is Tracking::Page.
      4668ea6b
    • azul's avatar
      acae4e8c
    • azul's avatar
      track_actions for all participation tracking · 4436e29d
      azul authored
      Allows us to remove UserParticipationObserver and GroupParticipationObserver.
      
      All changes to participations are now tracked by PageHistory::UpdateParticipation.
      This will test it's subclasses if they take care of the participation in question.
      The first subclass for which Class.tracks(changes, participation) returns true
      will be used.
      
      This allows us to handle different changes in different classes without a central
      registry. It requires however for all classes to be defined in the same file or
      the use of require_dependency directives. For now we still have all PageHistory
      subclasses in one file.
      4436e29d
  13. 16 Jun, 2015 1 commit
  14. 15 Jun, 2015 1 commit
  15. 12 Jun, 2015 1 commit
  16. 11 Jun, 2015 6 commits