Page MenuHomeDevCentral
Differential D1641

Allow to manipulate time ranges and check overlaps
ClosedPublic
Actions

Authored by dereckson on May 18 2018, 01:51.
Tags
None
Referenced Files
F9608863: D1641.diff
Mon, Jun 9, 23:13
F9604428: D1641.id.diff
Mon, Jun 9, 20:14
F9595094: D1641.id4189.diff
Mon, Jun 9, 13:52
F9594336: D1641.diff
Mon, Jun 9, 13:00
F9593063: D1641.id4190.diff
Mon, Jun 9, 11:30
Unknown Object (File)
Sun, Jun 8, 18:38
Unknown Object (File)
Sun, Jun 8, 15:51
Unknown Object (File)
Sun, Jun 8, 12:29
View All Files
Subscribers
None

Details

Reviewers
dereckson
Commits
rKOT6a9d07fcfe0e: Allow to manipulate time ranges and check overlaps
rKERUALD40d6f2ca3a04: Allow to manipulate time ranges and check overlaps
Test Plan

Unit tests provided.

Overlap test suite from https://codereview.stackexchange.com/questions/45784/test-2-time-ranges-to-see-if-they-overlap

Diff Detail

Repository
rKOT Keruald OmniTools
Lint
Lint Passed
Unit
Tests Passed
Branch
OverlappingTime
Build Status
Buildable 2575
Build 2823: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.May 18 2018, 01:51
dereckson created this revision.
Harbormaster completed remote builds in B2572: Diff 4186.May 18 2018, 01:51
dereckson changed the visibility from "Public (No Login Required)" to "All Users".May 18 2018, 01:52
dereckson updated this revision to Diff 4189.May 18 2018, 05:14

Allow HH:MM:SS format

Harbormaster completed remote builds in B2574: Diff 4189.May 18 2018, 05:14
dereckson updated this revision to Diff 4190.May 19 2018, 11:12

Explain why we provide those ranges.

Harbormaster completed remote builds in B2575: Diff 4190.May 19 2018, 11:12
dereckson added inline comments.Sep 9 2018, 09:02
src/DateTime/Time.php
48

static

src/DateTime/TimeRange.php
38

static

dereckson planned changes to this revision.Nov 10 2024, 11:03

Should be rebased and resubmitted against rKERUALD repository.

src/DateTime/Time.php
48

17h23m14s to parse too¿

dereckson changed the visibility from "All Users" to "Public (No Login Required)".Nov 10 2024, 11:04
dereckson updated this revision to Diff 9217.Nov 10 2024, 11:08

Rebased against current rKOT HEAD

Harbormaster completed remote builds in B5681: Diff 9217.Nov 10 2024, 11:08
dereckson updated this revision to Diff 9218.Nov 10 2024, 11:13

Moved to rKERUALD monorepo.

Harbormaster completed remote builds in B5682: Diff 9218.Nov 10 2024, 11:13
dereckson updated this revision to Diff 9221.Nov 10 2024, 11:57

static, count time range minutes, add tests, update to PHPUnit 10

Harbormaster completed remote builds in B5684: Diff 9221.Nov 10 2024, 11:57
dereckson updated this revision to Diff 9222.Nov 10 2024, 12:05

Simplify TimeRange::countMinutes

Harbormaster completed remote builds in B5685: Diff 9222.Nov 10 2024, 12:05
dereckson updated this revision to Diff 9223.Nov 10 2024, 12:10

Generic.Files.LineLength.MaxExceeded

Harbormaster completed remote builds in B5686: Diff 9223.Nov 10 2024, 12:10
dereckson accepted this revision.Nov 10 2024, 12:10
This revision is now accepted and ready to land.Nov 10 2024, 12:10
Closed by commit rKERUALD40d6f2ca3a04: Allow to manipulate time ranges and check overlaps (authored by dereckson). · Explain WhyNov 10 2024, 12:10
This revision was automatically updated to reflect the committed changes.
dereckson added a commit: rKERUALD40d6f2ca3a04: Allow to manipulate time ranges and check overlaps.
dereckson added a commit: rKOT6a9d07fcfe0e: Allow to manipulate time ranges and check overlaps.Nov 10 2024, 12:22

Revision Contents
Changeset List

PathSize
src/
DateTime/
130 lines
91 lines
tests/
DateTime/
93 lines
86 lines

Diff 4190

View Options

src/DateTime/Time.php

Loading...
View Options

src/DateTime/TimeRange.php

Loading...
View Options

tests/DateTime/TimeRangeTest.php

Loading...
View Options

tests/DateTime/TimeTest.php

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator