-
Notifications
You must be signed in to change notification settings - Fork 183
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
Version the class #21
Comments
+1 I'm all for this. Also an extra var/define/constant/whatever to make the class unique would be great. Many times I need to tweet or add something project-specific that doesn't make sense to send in as pull request. It would be very handy in that case to not have conflicts with multiple plugins that use the class. |
Version number added on 96f4373 @daankortenbach I am not sure what you meant, but doesn't the |
|
Okay got it. But if you have any idea to make the class unique, let me know :) |
Haha, you funny guy. Sorry dude, sometimes I have a brainfart that just doesn't make sense ;) (extend class with basename that calls specific class with version number in classname) |
My suggestion is to rename the class per plugin I use it with. this is what I do. this is to avoid any conflicts with other plugins that may use older or newer version. sometimes I also have small modifications in the class so this will also avoid problems to the other plugins in case my plugin loads first. |
Why would you want to load the class multiple times for multiple plugins? The cool thing about a class like this is that it can be loaded once and reused in different plugins. Now what I like to do is to use the latest version of the class no matter which plugins that are using it. |
because I know I have changes. you can use 1 name for all your plugins as you can track and update the class for all your plugins but I think it should still be different than the default from whats in the git. I try to submit the changes I do but for example I added input type url which if my class won't load there will be a problem and a bad behavior. |
So why don't you just rename your class? I really don't get your logic and the big problem here? Just change line 13 in the main php file. |
There is no problem. That's what I said. I renamed the class. On Thu, Apr 9, 2015, 20:55 ahansson89 notifications@github.com wrote:
|
Well, then it is two different use cases. For other people using this class, without any modifications, they want the WP admin to load the latest version of the class no matter which plugin is using it. This is why I opened this ticket. |
Just added initial composer support (see readme), so you can stay updated. Haven't added the version number yet. Also updated the directory structure. Added changelog too |
Right now I am using this class for multiple projects and I would love if I could see what version my class have compared to the version on github.
The text was updated successfully, but these errors were encountered: