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

Optional enable/disable status #38

Closed
maderlock opened this issue Aug 15, 2014 · 2 comments
Closed

Optional enable/disable status #38

maderlock opened this issue Aug 15, 2014 · 2 comments

Comments

@maderlock
Copy link

When creating a bunch of entities, only one of which is the front-end entity, it tends to only make sense for that single front-end entity to have enable/disable as an option. Along with the mandatory name field, it is the enable/disable status that I find myself having to manually strip out from most entities. Didn't this use to be an option?

@tzyganu
Copy link
Owner

tzyganu commented Aug 15, 2014

@maderlock. I guess you are right. It does make sense to have the status field when you have frontend listings also.This also used to be an option in the older versions. I removed that option because It became hard to manage when generating the files for the extensions. I had a lot of if statements and a lot of different cases to tackle when the status field was optional. So I decide it to make it a system field. In case you don't need it when you have a module that affects only the backend all you need to do is to remove it from the install script and from the admin add/edit form and grid, before installing the module.
Sorry for disappointing you but I have no plans in the near future to make this optional again.
Like I said, to much work needed for something that can be removed easily after the module creation.

I will leave this open though, in case I have a sudden change of heart. 😄

@maderlock
Copy link
Author

Thanks for the reply. I understand the decision to leave it out due to complexity. Having a system option seems a reasonable compromise.

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

No branches or pull requests

2 participants