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

Component: Throw exception in case of signal method in Component or Presenter is private #263

Closed
wants to merge 21 commits into from

Conversation

janbarasek
Copy link
Contributor

  • new feature
  • BC break? yes

I think for better debuging is better throw exception in case of method is private.

I don't think there are many cases where I want the method name to be a valid signal and yet it's private.

dg and others added 21 commits June 9, 2020 14:54
Automatically will register methods as latte custom functions
@dg dg force-pushed the master branch 4 times, most recently from ea5ac15 to 0e37ebd Compare November 16, 2020 01:42
@dg dg force-pushed the master branch 2 times, most recently from af99b74 to ad60129 Compare November 25, 2020 18:17
@dg dg force-pushed the master branch 3 times, most recently from cfac883 to 45d9ace Compare December 17, 2020 22:27
@dg dg force-pushed the master branch 5 times, most recently from 3fb89c6 to fcb2578 Compare December 31, 2020 18:40
@dg dg force-pushed the master branch 3 times, most recently from 3c5786c to f84c16a Compare January 22, 2021 16:42
@dg dg force-pushed the master branch 5 times, most recently from 13f5ab3 to c15bd94 Compare January 26, 2021 21:24
@dg dg force-pushed the master branch 3 times, most recently from b3dcfb1 to de09883 Compare February 4, 2021 03:57
@dg dg closed this in 28f7840 Feb 5, 2021
dg pushed a commit that referenced this pull request Feb 5, 2021
dg pushed a commit that referenced this pull request Feb 5, 2021
dg pushed a commit that referenced this pull request Feb 8, 2021
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.

3 participants