[PR #390] [CLOSED] Switch to Caxy\HtmlDiff to drop the tidy requirement #5643

New Issue

OVERLORD · 2026-02-05T10:13:07+03:00

OVERLORD commented

2026-02-05 10:13:07 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/390
Author: @mlantz
Created: 5/9/2017
Status: ❌ Closed

Base: master ← Head: drop_tidy

📝 Commits (1)

fb687e0 Drop the tidy extension

📊 Changes

3 files changed (+374 additions, -307 deletions)

View changed files

📝 app/Http/Controllers/PageController.php (+6 -4)
📝 composer.json (+2 -3)
📝 composer.lock (+366 -300)

📄 Description

Using this package provides the HTML diff while avoiding the need for the tidy requirement which is not present in homestead or other default server setups. Concerns issue #381

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/390 **Author:** [@mlantz](https://github.com/mlantz) **Created:** 5/9/2017 **Status:** ❌ Closed **Base:** `master` ← **Head:** `drop_tidy` --- ### 📝 Commits (1) - [`fb687e0`](https://github.com/BookStackApp/BookStack/commit/fb687e00a541e204c86dd864ea48b508550be564) Drop the tidy extension ### 📊 Changes **3 files changed** (+374 additions, -307 deletions) <details> <summary>View changed files</summary> 📝 `app/Http/Controllers/PageController.php` (+6 -4) 📝 `composer.json` (+2 -3) 📝 `composer.lock` (+366 -300) </details> ### 📄 Description Using this package provides the HTML diff while avoiding the need for the tidy requirement which is not present in homestead or other default server setups. Concerns issue #381 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2026-02-05 10:13:07 +03:00

OVERLORD closed this issue

2026-02-05 10:13:10 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#5643