Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Port UserGroup to Use Eloquent Model #10506

Open
Hafsa-Naeem opened this issue Oct 4, 2024 · 3 comments
Open

Port UserGroup to Use Eloquent Model #10506

Hafsa-Naeem opened this issue Oct 4, 2024 · 3 comments
Assignees
@Hafsa-Naeem
Labels
Enhancement:2:Moderate A new feature or improvement that can be implemented in less than 4 weeks. Housekeeping:2:Urgent Any dependency management or refactor that must be done soon to fix or avoid a significant problem.
Milestone
3.5.0 LTS

Comments

@Hafsa-Naeem
Copy link
Contributor

Hafsa-Naeem commented Oct 4, 2024 •
edited by Vitaliy-1
Loading

Issue Description

The current implementation of the UserGroup does not utilize Eloquent. This limitation makes it hard to manage relationships, run complex queries efficiently and perform eager loading, especially when adding new fields to the Submission API (Issue #10480)

Proposed Solution

Refactor UserGroup class to use Eloquent Model and define appropriate Eloquent relationships. This will enable the use of eager loading (StageAssignment::with('userGroup')) to optimize data retrieval, reduce the number of database queries, and ensure accurate computation of the additional fields required for the Submission API.

PRs
pkp-lib: #10667
Citation Style Language Plugin: pkp/citationStyleLanguage#137
OAI JATS: pkp/oaiJats#61
Web Feed: pkp/webFeed#10
OJS: pkp/ojs#4547
OMP: pkp/omp#1787
OPS: pkp/ops#827
@Hafsa-Naeem Hafsa-Naeem added the Accessibility Any issue that impacts assistive technology or users with visual or physical impairments. label Oct 4, 2024
@Hafsa-Naeem Hafsa-Naeem self-assigned this Oct 4, 2024
@Hafsa-Naeem Hafsa-Naeem added Enhancement:2:Moderate A new feature or improvement that can be implemented in less than 4 weeks. and removed Accessibility Any issue that impacts assistive technology or users with visual or physical impairments. labels Oct 4, 2024
@jardakotesovec jardakotesovec added this to the 3.5.0 LTS milestone Oct 7, 2024
@Vitaliy-1 Vitaliy-1 added the Housekeeping:2:Urgent Any dependency management or refactor that must be done soon to fix or avoid a significant problem. label Oct 10, 2024
Hafsa-Naeem added a commit to Hafsa-Naeem/pkp-lib that referenced this issue Oct 15, 2024
@Hafsa-Naeem
pkp#10506 Refactor UserGroup to use Eloquent
35b9299
@Hafsa-Naeem
Copy link
Contributor Author

@Vitaliy-1 I've submitted a draft PR as discussed in the meeting. Could you please review it and share your feedback so I continue with the commits for usage and migrations?
pkp/pkp-lib pkp/pkp-lib#10506 Refactor UserGroup to use Eloquent #10519

Hafsa-Naeem added a commit to Hafsa-Naeem/pkp-lib that referenced this issue Oct 28, 2024
@Hafsa-Naeem
pkp#10506 Update usage for porting UserGroup to Eloquent
f20097d
Hafsa-Naeem added a commit to Hafsa-Naeem/ojs that referenced this issue Oct 28, 2024
@Hafsa-Naeem
pkp/pkp-lib#10506 Update usage for porting UserGroup to Eloquent
2e5a05b
@Hafsa-Naeem
Copy link
Contributor Author

@Vitaliy-1 I've made changes according to our discussion in the meeting and also updated usage part.
pkp/pkp-lib pkp/pkp-lib#10506 Refactor UserGroup to use Eloquent #10519

Hafsa-Naeem added a commit to Hafsa-Naeem/pkp-lib that referenced this issue Nov 15, 2024
@Hafsa-Naeem
pkp#10506 Fix UserGroup and Repository code to correct settings handl…
1ce4b3f 
…ing and Eloquent conventions
Hafsa-Naeem added a commit to Hafsa-Naeem/webFeed that referenced this issue Nov 18, 2024
@Hafsa-Naeem
pkp/pkp-lib#10506 usage update after porting UserGroup to Eloquent
9c35c3b
Hafsa-Naeem added a commit to Hafsa-Naeem/ojs that referenced this issue Nov 18, 2024
@Hafsa-Naeem
pkp/pkp-lib#10506 usage update after porting UserGroup to Eloquent
ba596b3
Hafsa-Naeem added a commit to Hafsa-Naeem/ojs that referenced this issue Nov 18, 2024
@Hafsa-Naeem
pkp/pkp-lib#10506 usage update after porting UserGroup to Eloquent
0a46028
Hafsa-Naeem added a commit to Hafsa-Naeem/citationStyleLanguage that referenced this issue Nov 18, 2024
@Hafsa-Naeem
pkp/pkp-lib#10506 usage update after porting UserGroup to Eloquent
785b413
Hafsa-Naeem added a commit to Hafsa-Naeem/pkp-lib that referenced this issue Nov 18, 2024
@Hafsa-Naeem
pkp#10506 optimize relationships with eager loading
2ef27fa
@Hafsa-Naeem Hafsa-Naeem mentioned this issue Nov 18, 2024
Draft
Hafsa-Naeem added a commit to Hafsa-Naeem/ojs that referenced this issue Nov 22, 2024
@Hafsa-Naeem
pkp/pkp-lib#10506 submodules updates, use Eloquent for UserGroup
5795fa9
@Hafsa-Naeem
Copy link
Contributor Author

@Vitaliy-1 Ready for review again
pkp-lib pkp/pkp-lib#10506 Refactor UserGroup to use Eloquent #10519

Vitaliy-1 pushed a commit to Vitaliy-1/pkp-lib that referenced this issue Nov 26, 2024
@Hafsa-Naeem @Vitaliy-1
pkp#10506 Refactor UserGroup to use Eloquent
8eb0100
Vitaliy-1 pushed a commit to Vitaliy-1/pkp-lib that referenced this issue Nov 26, 2024
@Hafsa-Naeem @Vitaliy-1
pkp#10506 Refactor UserGroup to use Eloquent
e2f64a2
Vitaliy-1 pushed a commit to Vitaliy-1/ojs that referenced this issue Nov 27, 2024
@Hafsa-Naeem @Vitaliy-1
pkp/pkp-lib#10506 usage update after porting UserGroup to Eloquent
31d7cb3
Vitaliy-1 added a commit to Vitaliy-1/pkp-lib that referenced this issue Nov 27, 2024
@Vitaliy-1
pkp#10506 Fix setData() method and snake case usage
78ddb16
Vitaliy-1 added a commit to Vitaliy-1/pkp-lib that referenced this issue Nov 27, 2024
@Vitaliy-1
pkp#10506 Fix call to a removed User Group Collector
3095bbc
Vitaliy-1 added a commit to Vitaliy-1/pkp-lib that referenced this issue Nov 29, 2024
@Vitaliy-1
pkp#10506 Fix User Group filter by context IDs
088c8d7
Vitaliy-1 added a commit to Vitaliy-1/pkp-lib that referenced this issue Nov 29, 2024
@Vitaliy-1
pkp#10506 Fix calls to removed user group collector
5fd81a3
Vitaliy-1 pushed a commit to Vitaliy-1/pkp-lib that referenced this issue Nov 29, 2024
@Hafsa-Naeem @Vitaliy-1
pkp#10506 Refactor UserGroup to use Eloquent
82a3050
Vitaliy-1 added a commit to Vitaliy-1/pkp-lib that referenced this issue Nov 29, 2024
@Vitaliy-1
pkp#10506 Fix setData() method and snake case usage
41c503d
Vitaliy-1 added a commit to Vitaliy-1/pkp-lib that referenced this issue Nov 29, 2024
@Vitaliy-1
pkp#10506 Fix call to a removed User Group Collector
0bc8342
Vitaliy-1 added a commit to Vitaliy-1/pkp-lib that referenced this issue Dec 14, 2024
@Vitaliy-1
pkp#10506 Fix User isn't a Model and tests
67618b2
Vitaliy-1 added a commit to Vitaliy-1/pkp-lib that referenced this issue Dec 14, 2024
@Vitaliy-1
pkp#10506 Fix native import export plugin
fe42c1a
Vitaliy-1 added a commit to Vitaliy-1/ojs that referenced this issue Dec 14, 2024
@Vitaliy-1
pkp/pkp-lib#10506 Refactor User Group to Eloquent Model
a736c1a
Vitaliy-1 added a commit to Vitaliy-1/ojs that referenced this issue Dec 14, 2024
@Vitaliy-1
pkp/pkp-lib#10506 Fix subscriber select error
9555e9e
Vitaliy-1 added a commit to Vitaliy-1/ojs that referenced this issue Dec 14, 2024
@Vitaliy-1
pkp/pkp-lib#10506 Turn User Group to a Collection
e72c46b
Vitaliy-1 added a commit to Vitaliy-1/omp that referenced this issue Dec 14, 2024
@Vitaliy-1
pkp/pkp-lib#10506 User Group refactoring to Eloquent Model
883f65d
Vitaliy-1 added a commit to Vitaliy-1/ops that referenced this issue Dec 14, 2024
@Vitaliy-1
pkp/pkp-lib#10506 Refactor User Group to Eloquent Model
626c573
Vitaliy-1 added a commit to Vitaliy-1/ops that referenced this issue Dec 14, 2024
@Vitaliy-1
pkp/pkp-lib#10506 Refactor User Group to Eloquent Model
0ee1c71
Vitaliy-1 added a commit to Vitaliy-1/pkp-lib that referenced this issue Dec 15, 2024
@Vitaliy-1
pkp#10506 Fix UserGroup grid handler
d011a7a
Vitaliy-1 added a commit to Vitaliy-1/pkp-lib that referenced this issue Dec 15, 2024
@Vitaliy-1
pkp#10506 UserGroup LazyCollection to Collection
d05f1af
Vitaliy-1 added a commit to Vitaliy-1/omp that referenced this issue Dec 15, 2024
@Vitaliy-1
pkp/pkp-lib#10506 User Group refactoring to Eloquent Model
3ad25f5
Vitaliy-1 added a commit to Vitaliy-1/omp that referenced this issue Dec 15, 2024
@Vitaliy-1
pkp/pkp-lib#10506 User Group refactoring to Eloquent Model
0a0e6b8
Vitaliy-1 added a commit to Vitaliy-1/ops that referenced this issue Dec 15, 2024
@Vitaliy-1
pkp/pkp-lib#10506 Refactor User Group to Eloquent Model
494353b
Vitaliy-1 added a commit to Vitaliy-1/omp that referenced this issue Dec 16, 2024
@Vitaliy-1
pkp/pkp-lib#10506 User Group refactoring to Eloquent Model
6f38de6
Vitaliy-1 added a commit to Vitaliy-1/omp that referenced this issue Dec 16, 2024
@Vitaliy-1
pkp/pkp-lib#10506 User Group refactoring to Eloquent Model
5a75e27
Vitaliy-1 added a commit to Vitaliy-1/omp that referenced this issue Dec 16, 2024
@Vitaliy-1
pkp/pkp-lib#10506 User Group refactoring to Eloquent Model
6fcbd6c
Vitaliy-1 added a commit to Vitaliy-1/ojs that referenced this issue Dec 16, 2024
@Vitaliy-1
pkp/pkp-lib#10506 Refactor User Group to Eloquent Model
1ac0dcd
Vitaliy-1 added a commit to Vitaliy-1/ojs that referenced this issue Dec 16, 2024
@Vitaliy-1
pkp/pkp-lib#10506 Fix subscriber select error
03abd49
Vitaliy-1 added a commit to Vitaliy-1/ojs that referenced this issue Dec 16, 2024
@Vitaliy-1
pkp/pkp-lib#10506 Turn User Group to a Collection
17fc029
Vitaliy-1 added a commit that referenced this issue Dec 16, 2024
@Vitaliy-1
Merge pull request #10667 from Vitaliy-1/i10506_usergroup_eloquent
b0b3fed 
#10506 user group refactoring to Eloquent
Vitaliy-1 added a commit to pkp/citationStyleLanguage that referenced this issue Dec 16, 2024
@Vitaliy-1
Merge pull request #137 from Vitaliy-1/i10506_usergroup_eloquent
c7e90fa 
pkp/pkp-lib#10506 Refactor UserGroup to Eloquent Model
Vitaliy-1 added a commit to pkp/webFeed that referenced this issue Dec 16, 2024
@Vitaliy-1
Merge pull request #10 from Vitaliy-1/i10506_usergroup_eloquent
1361507 
pkp/pkp-lib#10506 User Group refactoring to Eloquent Model
Vitaliy-1 added a commit to pkp/ojs that referenced this issue Dec 16, 2024
@Vitaliy-1
Merge pull request #4547 from Vitaliy-1/i10506_usergroup_eloquent
e3069d7 
pkp/pkp-lib#10506 User Group refactoring to Eloquent
Vitaliy-1 added a commit to pkp/omp that referenced this issue Dec 16, 2024
@Vitaliy-1
Merge pull request #1787 from Vitaliy-1/i10506_usergroup_eloquent
74b1370 
pkp/pkp-lib#10506 User Group refactoring to Eloquent Model
Vitaliy-1 added a commit to pkp/ops that referenced this issue Dec 16, 2024
@Vitaliy-1
Merge pull request #827 from Vitaliy-1/i10506_usergroup_eloquent
2fb914a 
pkp/pkp-lib#10506 Refactor UserGroup to Eloquent Model
asmecher pushed a commit to pkp/oaiJats that referenced this issue Dec 16, 2024
@Vitaliy-1
pkp/pkp-lib#10506 User Group refactoring to Eloquent Model (#61)
51bf1b3
jardakotesovec added a commit to pkp/ui-library that referenced this issue Dec 18, 2024
@jardakotesovec
pkp/pkp-lib#10506 Adjust Participant UI for small data structure change
0ae6ff9
This was referenced Dec 24, 2024
Closed
Closed
@touhidurabir touhidurabir mentioned this issue Jan 15, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Hafsa-Naeem Hafsa-Naeem

Labels
Enhancement:2:Moderate A new feature or improvement that can be implemented in less than 4 weeks. Housekeeping:2:Urgent Any dependency management or refactor that must be done soon to fix or avoid a significant problem.
Projects
None yet
Milestone
3.5.0 LTS
Development

No branches or pull requests
3 participants
@jardakotesovec @Vitaliy-1 @Hafsa-Naeem