Skip to content
This repository has been archived by the owner on Nov 9, 2024. It is now read-only.

Remove private fields from mixins #4

Open
kerwanp opened this issue Oct 7, 2024 · 0 comments
Open

Remove private fields from mixins #4

kerwanp opened this issue Oct 7, 2024 · 0 comments
Labels
enhancement New feature or request
Milestone

Comments

@kerwanp
Copy link
Contributor

kerwanp commented Oct 7, 2024

Description

Private properties are not allowed in mixins and private fields (ex: #create()) result in a strange behavior.
image

Proposal

Mixins should never have any private field. They must be either protected or public and when the usage is "internal only" they must be prefixed with $.

class ManagesCustomer {
  publicMethod() {}
  $internalMethod() ()
}
@kerwanp kerwanp added the enhancement New feature or request label Oct 7, 2024
@kerwanp kerwanp added this to the v1 milestone Oct 7, 2024
@kerwanp kerwanp mentioned this issue Oct 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant