# [PATCH] tidylib behaviour changed

- Thread: https://externals.io/message/10905
- Original archive: https://news-web.php.net/php.internals/10905
- Started by Nuno Lopes on 2004-07-05T09:47:44+00:00
- Messages: 2
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #10905 — Nuno Lopes — 2004-07-05T09:47:44+00:00

URL: https://externals.io/message/10905#10905

Hi,

Finally today, the tidy team has corrected a bug in tidyLoadConfig. Attached
is a patch that fixes PHP for the new behaviour (which is backward
compatible with older libs) and I've also removed the repairString() and
repairFile() methods (see why in my previous e-mail).

Regards,
Nuno



--------------------------------------------------
Tidy bug:
http://sourceforge.net/tracker/?func=detail&atid=390963&aid=937271&group_id=27659

Tidy response:
"I have changed your patch, so that the tidyLoadConfig() return value is
consistent with the existing usage of tidyLoadConfig() - e.g.
in "tidy.c":

-1 if the file is not found
0 if the file was parsed correctly
1 if there were errors parsing the file (unknown option, etc...)

i.e. no change to the last line of the function."

---

## #10907 — Steph — 2004-07-05T12:44:29+00:00

URL: https://externals.io/message/10905#10907 · In reply to: #10905

Those methods work fine.  The code's incorrect in your example; you were
echoing the original string.

> -----Original Message-----
> From: Nuno Lopes [mailto:nlopess@php.net]
> Sent: 05 July 2004 10:48
> To: PHPdev
> Subject: [PHP-DEV] [PATCH] tidylib behaviour changed
>
>
> Hi,
>
> Finally today, the tidy team has corrected a bug in
> tidyLoadConfig. Attached
> is a patch that fixes PHP for the new behaviour (which is backward
> compatible with older libs) and I've also removed the repairString() and
> repairFile() methods (see why in my previous e-mail).
>
> Regards,
> Nuno
>
>
>
> --------------------------------------------------
> Tidy bug:
> http://sourceforge.net/tracker/?func=detail&atid=390963&aid=937271
&group_id=27659

Tidy response:
"I have changed your patch, so that the tidyLoadConfig() return value is
consistent with the existing usage of tidyLoadConfig() - e.g.
in "tidy.c":

-1 if the file is not found
0 if the file was parsed correctly
1 if there were errors parsing the file (unknown option, etc...)

i.e. no change to the last line of the function."