Bug #69196 [Asn->Opn]: PharData should extends Phar

Edit report at https://bugs.php.net/bug.php?id=69196&edit=1 ID: 69196 Updated by: kalle@php.net Reported by: hywan@php.net Summary: PharData should extends Phar -Status: Assigned +Status: Open Type: Bug Package: PHAR related PHP Version: 5.6Git-2015-03-06 (Git) -Assigned To: reeze +Assigned To: Block user comment: N Private report: N Previous Comments: ------------------------------------------------------------------------ [2015-03-30 09:50:33] mike@php.net Sure, I'm with you, but I wouldn't fix implementation of "1+2 == 3" if documentation says "1+2 == 4". I just mean we should think about what is th right thing to do. ------------------------------------------------------------------------ [2015-03-30 03:26:20] reeze@php.net -- Repost. Hi Mike, The bug #69196 could be fixed by specify different arg info IMO. class Phar and PharData shared the most of the implementation, we could either fix doc or fix implementation, but as the documentation exist so long, I do prefer fix the implementation. ------------------------------------------------------------------------ [2015-03-30 03:26:20] reeze@php.net Related To: Bug #69196 ------------------------------------------------------------------------ [2015-03-30 02:23:33] reeze@php.net Related To: Bug #52909 ------------------------------------------------------------------------ [2015-03-29 07:13:51] mike@php.net Not sure it should really be this way (Phar extends PharData or PharData extends Phar). Not directly related: See also bug #52909 ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=69196 -- Edit this bug report at https://bugs.php.net/bug.php?id=69196&edit=1