Opened 3 years ago
Closed 3 years ago
#58217 closed defect (bug) (fixed)
The URL defined "Text_Diff_Engine_native" is not working as expected.
| Reported by: | utsav72640 | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.3 |
| Component: | External Libraries | Version: | 6.2 |
| Severity: | normal | Keywords: | has-patch changes-requested |
| Cc: | Focuses: | docs |
Description
The URL defined "Text_Diff_Engine_native" is not working as expected.
Path : wp-includes/Text/Diff/Engine/native.php
I have attached a list of URLs that returned a 404 error.
http://www.perl.com/CPAN/authors/id/N/NE/NEDKONZ/Algorithm-Diff-1.06.zip
http://opensource.org/licenses/lgpl-license.php
Attachments (6)
Change History (11)
#1
@
3 years ago
- Focuses docs added
@utsav72640 Thank you for the report
Due to the publication of a new licence version for the LGPL, I inspected those URLs and discovered that they all redirect to 404. I have added an attachment patch and updated it with a new URL.
Thanks
#2
@
3 years ago
- Keywords has-patch changes-requested added; dev-feedback removed
- Milestone Awaiting Review → 6.3
Hi there, thanks for the patch!
- Per the Text_Diff page on GitHub, it has been superseded by the Horde_Text_Diff package, which uses version 2.1 of the LGPL license, so https://opensource.org/license/lgpl-2-1/ would likely be the correct URL here as well, instead of the 3.0 version.
- The
Algorithm::DiffURL should still link to version 1.06 for accuracy, so either of these URLs would be correct:
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
wp-includes/Text/Diff/Engine/native.php