Skip to content

Conversation

@IanDelMar
Copy link
Contributor

@IanDelMar IanDelMar commented Nov 4, 2023

First steps towards what Viktor calls a "proper project".

#119 (comment)

Done:

  • Move all classes from inside visitor.php to separate files located in the "src/" directory.
  • Add tests to ensure that FunctionMap() additions are working correctly.

Todo:

  • Refactor Visitor.php
  • Move void and never detection outside of the visitor. For detecting void/never we parse the source. This needs to happen on source change only.
  • Investigate the potential advantage of utilising doc blocks for @phpstan-....

* Improve return type of get_comment() and get_post()

* Add wpdb::get_row() to functions map

* Add tests for wp_clear_scheduled_hook()

* Add wpdb::get_results() to the functions map

* Add get_bookmark() to function map

* Add get_category() to function map

* Add argument type to get_term()

* Use fully qualified name

* Add get_category_by_path() to function map

* Remove types already added without function map

* Add spaces around array assignment operator

* Regenerate function map

* Add trailing comma to last array item
Removes `getAdditionFromParam()`, `getAdditionFromReturn()` and getAdditionFromVar()`
@szepeviktor
Copy link
Member

Should I merge it and release 6.4?

@IanDelMar
Copy link
Contributor Author

I'd release 6.4 first.

@szepeviktor
Copy link
Member

Thank you, Safe Ian!

@szepeviktor
Copy link
Member

The 6.4 branch cannot be merged to master.
Please close this one and open a new PR to master.

Thank you for your contributions.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants