Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:10893
Return-Path: <nlopess@php.net>
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Delivered-To: mailing list internals@lists.php.net
Received: (qmail 4322 invoked by uid 1010); 4 Jul 2004 10:05:49 -0000
Delivered-To: ezmlm-scan-internals@lists.php.net
Delivered-To: ezmlm-internals@lists.php.net
Received: (qmail 4275 invoked from network); 4 Jul 2004 10:05:48 -0000
Received: from unknown (HELO sapo.pt) (212.55.154.24)
  by pb1.pair.com with SMTP; 4 Jul 2004 10:05:48 -0000
Received: (qmail 22321 invoked from network); 4 Jul 2004 10:05:47 -0000
Received: from unknown (HELO sapo.pt) (10.134.35.156)
  by relay4 with SMTP; 4 Jul 2004 10:05:47 -0000
Received: (qmail 6386 invoked from network); 4 Jul 2004 10:05:09 -0000
Received: from unknown (HELO pc07653) (nunoplopes@sapo.pt@[81.193.142.22])
          (envelope-sender <nlopess@php.net>)
          by mta6 (qmail-ldap-1.03) with SMTP
          for <internals@lists.php.net>; 4 Jul 2004 10:05:09 -0000
Message-ID: <005a01c461af$2e64de30$0100a8c0@pc07653>
To: "PHPdev" <internals@lists.php.net>
Date: Sun, 4 Jul 2004 11:10:49 +0100
MIME-Version: 1.0
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
X-Priority: 3
X-MSMail-Priority: Normal
X-Mailer: Microsoft Outlook Express 6.00.2800.1409
X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2800.1409
Subject: tidy->repairString and repairFile
From: nlopess@php.net ("Nuno Lopes")

Hello,

Today I was finishing writing the tidy docs and I've noted that the
functions $tidy->repairString() and $tidy->repairFile() simply don't work!
And I think these functions should be removed. I'll explain:

non-OO way:
<?
echo tidy_repair_string($html);
?>

OO way:
<?
$tidy = new tidy;
var_dump($tidy->repairString($html));
echo $html;

?>

The non-OO way is very simple, you give it a broken HTML and it returns a
repaired HTML string. And in OO? It returns nothing! And why should you want
that function in OO? tidy_repair_string() is a one-time use only function.
Even if it exists in OO, you can't do anything with it (even if it
worked...)

Can you delete those funtions, please (isn't really the functions, just the
OO mapping)?

Thanks,
Nuno