Skip to content
This repository has been archived by the owner on Dec 12, 2021. It is now read-only.

Issue with accessible_by and ActiveRecord SQL JOIN behavior #1002

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Issue with accessible_by and ActiveRecord SQL JOIN behavior #1002

wants to merge 8 commits into from

Commits on Apr 11, 2014

  1. removing the rvmrc file.. super out of date anyway

    @jackryon
    jackryon committed Apr 11, 2014
    Configuration menu
    Copy the full SHA
    094e75f View commit details
    Browse the repository at this point in the history

  2. swapping the joins call in active_record_adapted#database_records for… 

    … a call to includes. this yields a left outer join instead of an inner join. combined abilities now work for accessible_by call on a given model with complex hash conditions.
    @jackryon
    jackryon committed Apr 11, 2014
    Configuration menu
    Copy the full SHA
    50d8a85 View commit details
    Browse the repository at this point in the history

  3. prying

    @jackryon
    jackryon committed Apr 11, 2014
    Configuration menu
    Copy the full SHA
    d0f0479 View commit details
    Browse the repository at this point in the history

  4. puts statements for debugging

    @jackryon
    jackryon committed Apr 11, 2014
    Configuration menu
    Copy the full SHA
    df04d8d View commit details
    Browse the repository at this point in the history

  5. trying to debug this with some dum puts statements

    @jackryon
    jackryon committed Apr 11, 2014
    Configuration menu
    Copy the full SHA
    af6b9d7 View commit details
    Browse the repository at this point in the history

  6. jackjackjack

    @jackryon
    jackryon committed Apr 11, 2014
    Configuration menu
    Copy the full SHA
    ac59387 View commit details
    Browse the repository at this point in the history

  7. fooling around with gemfile.lock

    @jackryon
    jackryon committed Apr 11, 2014
    Configuration menu
    Copy the full SHA
    67e5093 View commit details
    Browse the repository at this point in the history

  8. removing debugging calls

    @jackryon
    jackryon committed Apr 11, 2014
    Configuration menu
    Copy the full SHA
    5e0a772 View commit details
    Browse the repository at this point in the history