Page MenuHomePhorge
Differential D25082

Remarkup: minor refactor to identify anchor and relative URIs
AbandonedPublic
Actions

Authored by valerio.bozzolan on Mar 23 2023, 09:20.
Tags
None
Referenced Files
F3330989: D25082.1743499019.diff
Mon, Mar 31, 09:16
F3330888: D25082.1743496275.diff
Mon, Mar 31, 08:31
F3320413: D25082.1743329261.diff
Sat, Mar 29, 10:07
F3318572: D25082.1743287743.diff
Fri, Mar 28, 22:35
F3309421: D25082.1743184301.diff
Thu, Mar 27, 17:51
F3308970: D25082.1743179977.diff
Thu, Mar 27, 16:39
F3308093: D25082.1743159289.diff
Thu, Mar 27, 10:54
F3307497: D25082.1743147323.diff
Thu, Mar 27, 07:35
View All 40 Files
Subscribers
Cigaryno
Matthew
speck
tobiaswiese

Details

Reviewers
None
Group Reviewers
O1: Blessed Committers
Maniphest Tasks
T15182: Simplify the way "internal links" are identified
Summary

This modification should not change anything logically.

This change simplifies these checks:

  • identify an anchor
  • identify a relative URI

https://we.phorge.it/T15182

The methods are introduced in PhutilRemarkupRule since that
is the parent class of these children classes:

  • PhutilRemarkupDocumentLinkRule
  • PhutilRemarkupHyperlinkRule

Ref T15182

Test Plan
  • Just check that nothing has been changed logically

Diff Detail

Repository
rP Phorge
Branch
T15161-remarkup-internal-links-refactor
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 155
Build 155: arc lint + arc unit

Event Timeline

valerio.bozzolan created this revision.Mar 23 2023, 09:20
Owners added a reviewer: O1: Blessed Committers.Mar 23 2023, 09:20
Herald added subscribers: Cigaryno, Matthew, tobiaswiese, speck. · View Herald TranscriptMar 23 2023, 09:20
valerio.bozzolan requested review of this revision.Mar 23 2023, 09:20
Harbormaster completed remote builds in B153: Diff 365.Mar 23 2023, 09:20
valerio.bozzolan removed a task: T15161: Internal resources should avoid to become external links, as default.Mar 23 2023, 09:21
valerio.bozzolan updated this revision to Diff 366.Mar 23 2023, 09:22
valerio.bozzolan added a task: T15161: Internal resources should avoid to become external links, as default.

run celerity to make lint happy

Harbormaster completed remote builds in B154: Diff 366.Mar 23 2023, 09:22
valerio.bozzolan updated this revision to Diff 367.Mar 23 2023, 09:24

improve inline doc a bit

Harbormaster completed remote builds in B155: Diff 367.Mar 23 2023, 09:24
valerio.bozzolan mentioned this in D25083: Remarkup: simplify the way "internal links" could be identified.Mar 23 2023, 10:08
valerio.bozzolan mentioned this in T15182: Simplify the way "internal links" are identified.Mar 23 2023, 10:11
valerio.bozzolan removed a task: T15161: Internal resources should avoid to become external links, as default.Mar 23 2023, 10:17
valerio.bozzolan abandoned this revision.Apr 7 2023, 21:16

Squashed here:

D25118: Remarkup: make less internal links open in new tabs

Revision Contents
Changeset List

PathSizeCoverage (All)Coverage (Touched)Packages
resources/
celerity/
100%-O1: Blessed Committers
src/
infrastructure/
markup/
markuprule/
4 lines1%
Loading...
O1: Blessed Committers
24 lines3%
Loading...
O1: Blessed Committers

Diff 367

View Options

resources/celerity/map.php

This file contains generated code, which does not normally need to be reviewed. Show File Contents
View Options

src/infrastructure/markup/markuprule/PhutilRemarkupDocumentLinkRule.php

This file contains generated code, which does not normally need to be reviewed. Show File Contents
View Options

src/infrastructure/markup/markuprule/PhutilRemarkupRule.php

This file contains generated code, which does not normally need to be reviewed. Show File Contents
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0