Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ReflectionClass::export incompatibilities #144

Closed
asgrim opened this issue Nov 2, 2015 · 4 comments
Closed

ReflectionClass::export incompatibilities #144

asgrim opened this issue Nov 2, 2015 · 4 comments

Comments

@asgrim
Copy link
Member

asgrim commented Nov 2, 2015

1) /home/travis/build/Roave/BetterReflection/test/compat/ReflectionClass_export_basic1.phpt
Failed asserting that format description matches text.
--- Expected
+++ Actual
@@ @@
 Class [ <user> class C extends A ] {
-  @@ %s 14-14
+  @@  3-7
@@ @@
   - Methods [2] {
-    Method [ <user, inherits A> public method privf ] {
-      @@ %s 3 - 3
-
+    Method [ <user> public method privf ] {
+      @@  6 - 8
+    
       - Parameters [1] {
         Parameter #0 [ <required> Exception $a ]
       }
     }
-
-    Method [ <user, inherits A> public method pubf ] {
-      @@ %s 4 - 11
-
+    Method [ <user> public method pubf ] {
+      @@  10 - 12
+    
       - Parameters [8] {
         Parameter #0 [ <required> A $a ]
         Parameter #1 [ <required> $b ]
         Parameter #2 [ <optional> C or NULL $c = NULL ]
-        Parameter #3 [ <optional> $d = '16 chars long -...' ]
+        Parameter #3 [ <optional> $d = '16 chars long --' ]
         Parameter #4 [ <optional> $e = '15 chars long -' ]
         Parameter #5 [ <optional> $f = NULL ]
         Parameter #6 [ <optional> $g = false ]
-        Parameter #7 [ <optional> array or NULL $h = NULL ]
+        Parameter #7 [ <optional> array $h = NULL ]
       }
     }
   }
 }
2) /home/travis/build/Roave/BetterReflection/test/compat/ReflectionClass_export_basic2.phpt
Failed asserting that format description matches text.
--- Expected
+++ Actual
@@ @@
 Class [ <user> class c ] {
-  @@ %s 2-5
+  @@  3-11
@@ @@
 }
-
 Class [ <user> class d extends c ] {
-  @@ %s 7-7
+  @@  3-7

   - Constants [0] {
   }

   - Static properties [0] {
   }

   - Static methods [0] {
   }

   - Properties [0] {
   }

   - Methods [0] {
   }
 }
@asgrim
Copy link
Member Author

asgrim commented Nov 2, 2015

Issues identified:

ReflectionClass_export_basic1.phpt

  • inherits definition is missing
  • string truncation is missing 16 chars long -...
  • array or NULL $h = NULL (correct) vs array $h = NULL (ours)

ReflectionClass_export_basic2.phpt

@asgrim asgrim added this to the 1.0.0 milestone Nov 2, 2015
@asgrim asgrim removed this from the 1.0.0 milestone Dec 21, 2015
@asgrim asgrim added the easypick label Feb 4, 2016
@Ocramius Ocramius added this to the 2.0.0 milestone Sep 8, 2017
@Ocramius
Copy link
Member

Ocramius commented Sep 8, 2017

Mostly handled in #361. Need to check whether anything still needs work here

@Ocramius Ocramius removed this from the 2.0.0 milestone Sep 8, 2017
@kukulich
Copy link
Collaborator

I think this can be closed. I've probably already implemented everything mentioned in #144 (comment). And export() methods were removed.

@Ocramius Ocramius added this to the 5.0.0 milestone Sep 23, 2021
@Ocramius Ocramius self-assigned this Sep 23, 2021
@Ocramius
Copy link
Member

Will mark it as done: ::export() is completely gone 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants