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.

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

42566 - fix checkbox across different pages #23075

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions app/controllers/correspondence_details_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -150,9 +150,12 @@ def save_correspondence_appeals
end
end
if params[:unselected_appeal_ids].present?
@correspondence.correspondence_appeals
correspondence_appeals_to_delete = @correspondence.correspondence_appeals
.where(appeal_id: params[:unselected_appeal_ids])
.delete_all

CorrespondencesAppealsTask.where(correspondence_appeal_id: correspondence_appeals_to_delete.pluck(:id)).delete_all

correspondence_appeals_to_delete.delete_all
end
respond_to do |format|
format.html
Expand Down
2 changes: 1 addition & 1 deletion app/models/correspondence_appeal.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,6 @@
class CorrespondenceAppeal < ApplicationRecord
belongs_to :correspondence
belongs_to :appeal
has_many :correspondences_appeals_tasks
has_many :correspondences_appeals_tasks, dependent: :destroy
has_many :tasks, through: :correspondences_appeals_tasks
end
3 changes: 2 additions & 1 deletion client/app/queue/CaseListTable.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ class CaseListTable extends React.PureComponent {
getKeyForRow = (rowNumber, object) => object.id;

toggleCheckboxState = (appealId) => {
const checked = this.props?.taskRelatedAppealIds.includes(Number(appealId));
const appealsToConsider = this.props?.initialAppealIds || this.props?.taskRelatedAppealIds;
const checked = appealsToConsider.includes(Number(appealId));

return checked ? this.props?.userAccess !== 'admin_access' : false;
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -452,7 +452,8 @@ const CorrespondenceDetails = (props) => {
appeals={appealsToDisplay}
paginate
showCheckboxes
taskRelatedAppealIds={disableSubmitButton ? selectedAppeals : initialSelectedAppeals}
taskRelatedAppealIds={selectedAppeals}
initialAppealIds={initialSelectedAppeals}
enableTopPagination
userAccess={userAccess}
checkboxOnChange={appealCheckboxOnChange}
Expand Down
Loading