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

Rename getImageList to getMIGX or similar #23

Open
gadamiak opened this issue Jun 24, 2011 · 3 comments
Open

Rename getImageList to getMIGX or similar #23

gadamiak opened this issue Jun 24, 2011 · 3 comments

Comments

@gadamiak
Copy link

At first, I need to say thank you for this incredibly useful extension! Now, I suggest renaming the getImageList snippet to getMIGX or similar but linking with MIGX itself. The current name is misleading -- my first thought was it's a fork of getResources.

@Bruno17
Copy link
Owner

Bruno17 commented Jun 25, 2011

The Name getImageList comes from my original custom-TV-input-type 'imageListTv' .
I thought about that to rename it. But I'm not sure, because renaming will probably break allready running installations.

@anselmh
Copy link

anselmh commented Jun 25, 2011

Think this should be postponed to a major release (2.0) when many things have changed that might be incompatible, too.

@gadamiak
Copy link
Author

That's a big change. Postponing it to a major release may be a good choice. Another name that came to my mind is getJSON as its quite generic snippet.

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

3 participants