Skip to content

Commit 7627182

Browse files
committed
Use updated JR branch which merges records on joins
1 parent fba3a64 commit 7627182

13 files changed

+21
-33
lines changed

Appraisals

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
appraise 'rails-4-2 pundit-1' do
22
gem 'rails', '4.2.0'
33
# ToDo: This is only for testing purposes
4-
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'track_join_options'
4+
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'use_records_for_joined_resources'
55
gem 'pundit', '~> 1.0'
66
group :development, :test do
77
gem 'sqlite3', '~> 1.3.13'
@@ -11,7 +11,7 @@ end
1111
appraise 'rails-5-0 pundit-1' do
1212
gem 'rails', '5.0.0'
1313
# ToDo: This is only for testing purposes
14-
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'track_join_options'
14+
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'use_records_for_joined_resources'
1515
gem 'pundit', '~> 1.0'
1616
group :development, :test do
1717
gem 'sqlite3', '~> 1.3.13'
@@ -21,7 +21,7 @@ end
2121
appraise 'rails-5-1 pundit-1' do
2222
gem "rails", "5.1.0"
2323
# ToDo: This is only for testing purposes
24-
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'track_join_options'
24+
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'use_records_for_joined_resources'
2525
gem 'pundit', '~> 1.0'
2626
group :development, :test do
2727
gem 'sqlite3', '~> 1.3.13'
@@ -31,7 +31,7 @@ end
3131
appraise 'rails-5-2 pundit-1' do
3232
gem 'rails', '5.2.0'
3333
# ToDo: This is only for testing purposes
34-
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'track_join_options'
34+
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'use_records_for_joined_resources'
3535
gem 'pundit', '~> 1.0'
3636
group :development, :test do
3737
gem 'sqlite3', '~> 1.3.13'
@@ -40,7 +40,7 @@ end
4040

4141
appraise 'rails-6-0 pundit-1' do
4242
gem 'rails', '~> 6.0.0'
43-
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'track_join_options'
43+
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'use_records_for_joined_resources'
4444
gem 'pundit', '~> 1.0'
4545
group :development, :test do
4646
gem 'sqlite3', '~> 1.4.1'
@@ -50,7 +50,7 @@ end
5050
appraise 'rails-4-2 pundit-2' do
5151
gem 'rails', '4.2.0'
5252
# ToDo: This is only for testing purposes
53-
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'track_join_options'
53+
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'use_records_for_joined_resources'
5454
gem 'pundit', '~> 2.0'
5555
group :development, :test do
5656
gem 'sqlite3', '~> 1.3.13'
@@ -60,7 +60,7 @@ end
6060
appraise 'rails-5-0 pundit-2' do
6161
gem 'rails', '5.0.0'
6262
# ToDo: This is only for testing purposes
63-
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'track_join_options'
63+
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'use_records_for_joined_resources'
6464
gem 'pundit', '~> 2.0'
6565
group :development, :test do
6666
gem 'sqlite3', '~> 1.3.13'
@@ -70,7 +70,7 @@ end
7070
appraise 'rails-5-1 pundit-2' do
7171
gem 'rails', '5.1.0'
7272
# ToDo: This is only for testing purposes
73-
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'track_join_options'
73+
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'use_records_for_joined_resources'
7474
gem 'pundit', '~> 2.0'
7575
group :development, :test do
7676
gem 'sqlite3', '~> 1.3.13'
@@ -80,7 +80,7 @@ end
8080
appraise 'rails-5-2 pundit-2' do
8181
gem 'rails', '5.2.0'
8282
# ToDo: This is only for testing purposes
83-
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'track_join_options'
83+
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'use_records_for_joined_resources'
8484
gem 'pundit', '~> 2.0'
8585
group :development, :test do
8686
gem 'sqlite3', '~> 1.3.13'
@@ -89,7 +89,7 @@ end
8989

9090
appraise 'rails-6-0 pundit-2' do
9191
gem 'rails', '~> 6.0.0'
92-
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'track_join_options'
92+
gem 'jsonapi-resources', :git => 'https://github.com/cerebris/jsonapi-resources.git', :branch => 'use_records_for_joined_resources'
9393
gem 'pundit', '~> 2.0'
9494
group :development, :test do
9595
gem 'sqlite3', '~> 1.4.1'

Gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ source 'https://rubygems.org'
44
gem(
55
'jsonapi-resources',
66
git: 'https://github.com/cerebris/jsonapi-resources.git',
7-
branch: 'track_join_options'
7+
branch: 'use_records_for_joined_resources'
88
)
99

1010
gemspec

gemfiles/rails_4_2_pundit_1.gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
source "https://rubygems.org"
44

5-
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "track_join_options"
5+
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "use_records_for_joined_resources"
66
gem "rails", "4.2.0"
77
gem "pundit", "~> 1.0"
88

gemfiles/rails_4_2_pundit_2.gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
source "https://rubygems.org"
44

5-
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "track_join_options"
5+
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "use_records_for_joined_resources"
66
gem "rails", "4.2.0"
77
gem "pundit", "~> 2.0"
88

gemfiles/rails_5_0_pundit_1.gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
source "https://rubygems.org"
44

5-
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "track_join_options"
5+
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "use_records_for_joined_resources"
66
gem "rails", "5.0.0"
77
gem "pundit", "~> 1.0"
88

gemfiles/rails_5_0_pundit_2.gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
source "https://rubygems.org"
44

5-
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "track_join_options"
5+
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "use_records_for_joined_resources"
66
gem "rails", "5.0.0"
77
gem "pundit", "~> 2.0"
88

gemfiles/rails_5_1_pundit_1.gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
source "https://rubygems.org"
44

5-
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "track_join_options"
5+
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "use_records_for_joined_resources"
66
gem "rails", "5.1.0"
77
gem "pundit", "~> 1.0"
88

gemfiles/rails_5_1_pundit_2.gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
source "https://rubygems.org"
44

5-
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "track_join_options"
5+
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "use_records_for_joined_resources"
66
gem "rails", "5.1.0"
77
gem "pundit", "~> 2.0"
88

gemfiles/rails_5_2_pundit_1.gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
source "https://rubygems.org"
44

5-
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "track_join_options"
5+
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "use_records_for_joined_resources"
66
gem "rails", "5.2.0"
77
gem "pundit", "~> 1.0"
88

gemfiles/rails_5_2_pundit_2.gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
source "https://rubygems.org"
44

5-
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "track_join_options"
5+
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "use_records_for_joined_resources"
66
gem "rails", "5.2.0"
77
gem "pundit", "~> 2.0"
88

gemfiles/rails_6_0_pundit_1.gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
source "https://rubygems.org"
44

5-
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "track_join_options"
5+
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "use_records_for_joined_resources"
66
gem "rails", "~> 6.0.0"
77
gem "pundit", "~> 1.0"
88

gemfiles/rails_6_0_pundit_2.gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
source "https://rubygems.org"
44

5-
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "track_join_options"
5+
gem "jsonapi-resources", git: "https://github.com/cerebris/jsonapi-resources.git", branch: "use_records_for_joined_resources"
66
gem "rails", "~> 6.0.0"
77
gem "pundit", "~> 2.0"
88

lib/jsonapi/authorization/pundit_scoped_resource.rb

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -10,18 +10,6 @@ def records(options = {})
1010
user_context = JSONAPI::Authorization.configuration.user_context(options[:context])
1111
::Pundit.policy_scope!(user_context, super)
1212
end
13-
14-
def apply_joins(records, join_manager, options)
15-
records = super
16-
join_manager.join_details.each do |k, v|
17-
next if k == '' || v[:join_type] == :root
18-
v[:join_options][:relationship_details][:resource_klasses].each_key do |klass|
19-
next unless klass.included_modules.include?(PunditScopedResource)
20-
records = records.where(v[:alias] => { klass._primary_key => klass.records(options)})
21-
end
22-
end
23-
records
24-
end
2513
end
2614
end
2715
end

0 commit comments

Comments
 (0)