Simplify tracking of URLs
authorTim Otten <totten@civicrm.org>
Thu, 14 Jan 2021 10:29:49 +0000 (02:29 -0800)
committerTim Otten <totten@civicrm.org>
Thu, 14 Jan 2021 10:35:06 +0000 (02:35 -0800)
commit6ad859f1b83295522d94e5392dd1cb00d5cd3775
tree2af9b9ae57fd7a30cf1968ae141b4b1511204fe1
parentba75e300ac0de014c1fdbc5bcf2caad4754bb998
Simplify tracking of URLs

In the prior commit, it needed to conceptually change the behavior of
`getTrackerURL($oldURL): $trackedURL`, but it danced around it (adding
more classes and similar-sounding-methods, but distributed elsewhere).

This re-consolidates to keep a cleaner separation of responsibilities
between `{Html,Text}ClickTracker.php` (*parsing an HTML or text message for URLS*)
and `TrackableURL.php` (*translating the URL*).
CRM/Mailing/BAO/TrackableURL.php
ext/flexmailer/src/ClickTracker/BaseClickTracker.php [deleted file]
ext/flexmailer/src/ClickTracker/HtmlClickTracker.php
ext/flexmailer/src/ClickTracker/TextClickTracker.php
ext/flexmailer/tests/phpunit/Civi/FlexMailer/ClickTrackerTest.php
ext/flexmailer/tests/phpunit/Civi/FlexMailer/FlexMailerSystemTest.php