Skip to content

[25] Add holiday adjustment#78

Open
ncouvert wants to merge 1 commit into
mainfrom
nco/enh/working-days
Open

[25] Add holiday adjustment#78
ncouvert wants to merge 1 commit into
mainfrom
nco/enh/working-days

Conversation

@ncouvert

@ncouvert ncouvert commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Issue: #25

Bug: Tasks with dependencies may incorrectly extend indefinitely. If a task is repeatedly moved backward by less than two days and this causes the end date of one of its dependencies to fall on a weekend, the dependent task may keep growing on each recalculation.

@ncouvert ncouvert force-pushed the nco/enh/working-days branch from d03f864 to f815a2f Compare July 1, 2026 11:03
@ncouvert ncouvert marked this pull request as draft July 1, 2026 11:47
@ncouvert ncouvert force-pushed the nco/enh/working-days branch from f815a2f to 6234af8 Compare July 1, 2026 12:41
@ncouvert ncouvert marked this pull request as ready for review July 1, 2026 13:54
@ncouvert ncouvert linked an issue Jul 1, 2026 that may be closed by this pull request
@ncouvert ncouvert force-pushed the nco/enh/working-days branch from 6234af8 to 499ce91 Compare July 1, 2026 13:58
@ncouvert

ncouvert commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator Author

This PR introduces changes to PepperMMJavaService.java, which is used by both the Details and Gantt views.

It is still possible to create a task that starts or ends on a weekend from the Explorer view.

I'm currently working on this issue, but reusing the business logic implemented in PepperMMJavaService.java would introduce a cyclic dependency between the Pepper Starter and Pepper Edit modules.

This will require some refactoring to resolve.

Issue: #25

Bug: Tasks with dependencies may incorrectly extend indefinitely. If a task is repeatedly moved backward by less than two days and this causes the end date of one of its dependencies to fall on a weekend, the dependent task may keep growing on each recalculation.
@ncouvert ncouvert force-pushed the nco/enh/working-days branch from 8c0493c to 10526f0 Compare July 3, 2026 09:58
@lfasani

lfasani commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Proposal:

  • The computation of a task that has dependencies should keep at most the logic the constraints of the tasks
    • If the task is Start-DURATION or End-Duration, the task should have its duration kept taking into account the holidays
    • If the task is Start-End, only the impacted date should be recomputed

As an enhancement, we could imagine having visual information to know if a task is constraint by its duration or not

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants