ID:               28261
 Comment by:       taufiq at krimnet dot com
 Reported By:      Philippe dot Jausions at 11abacus dot com
 Status:           Open
 Bug Type:         Feature/Change Request
 Operating System: *
 PHP Version:      6CVS, 5CVS
 New Comment:

I need this bug to be resolved.

I'm writing Javascript/CSS collector & minify library. 

I would like to code like below.

JS::include(FILE_PATH)->include(FILE_PATH2)->include(FILE_PATH3);

having method name other than include() is pretty annoying.


Previous Comments:
------------------------------------------------------------------------

[2007-12-16 00:46:09] kentfredric at gmail dot com

At the moment (5.2.3 ) this is perfectly valid.

Class A{
  function __call( $function, $args ){
     if( $function == 'print' ){
        print "MyPrint: {$args[0]}";
      }
   }
}
$a = new A();
$a->print( "hello" ); #<-- surprisingly, this is not an invalid use of
a keyword to the lexer. 
  # >> MyPrint: hello

but this

Class A{
  function print( $args ){
        print "MyPrint: {$args}";
   }
}
$a = new A();
$a->print( "hello" );

Yields a parse error "Parse error: syntax error, unexpected T_PRINT,
expecting T_STRING"

which appears to be an illogical design contstraint. 

I've seen rather brutal slander for people attempting to perform this (
#14178 , this bug ) amounting to "hey, you suck, dont do that" without
any rational explanation.

So yes, I look forward to this feature being integrated.

------------------------------------------------------------------------

[2007-10-05 02:13:49] cel...@php.net

as new reserved words are introduced, they tend to clash with existing
class's method names.  import and namespace are particularly nasty
examples of methods that are likely to clash, although I have also run
into problems with "list" which is a very nice method name for many
tasks.

The patches linked to in this comment provides a simple and effective
means of allowing reserved words in method names.  Not only is it
possible, but it is quite elegant :).

It also fixes, as a side effect, a bug in the parsing of this code:

<?php
class A {
var $list;
}
$a = new A;
$a->list = 1;
$a-> list = 2; // parse error, unexpected T_LIST
?>

whitespace between T_OBJECT_OPERATOR and the variable name changes the
token returned from T_STRING to any valid token.

Patch for PHP 5:
http://pear.php.net/~greg/smarter_lexer.patch.txt
Patch for PHP 6:
http://pear.php.net/~greg/php6_smarter_lexer.patch.txt

------------------------------------------------------------------------

[2004-05-04 18:18:05] he...@php.net

That's simply impossible.

------------------------------------------------------------------------

[2004-05-03 17:36:00] Philippe dot Jausions at 11abacus dot com

Description:
------------
Suggestion:

Wouldn't it be possible to lift the reserved keyword restriction for
method names?

It seems to me that there shouldn't be any namespace conflict with the
core PHP language.

Reproduce code:
---------------
<?php

class a {
   function eval() {
      // Do something...
   }
}

?>

Actual result:
--------------
Parse error: parse error, unexpected T_EVAL, expecting T_STRING


------------------------------------------------------------------------


-- 
Edit this bug report at http://bugs.php.net/?id=28261&edit=1

Reply via email to