-
Notifications
You must be signed in to change notification settings - Fork 30
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
Adds SemanticFormsSelect extension #248
Conversation
🐳 The image based on 9017fb93 commit has been built with |
@vedmaka - what feature(s) of SemanticFormsSelect do you think are useful, that are not already in Page Forms? Ironically, you yourself added some SMW querying syntax to Page Forms a few months ago, which I thought made SFS much less necessary. |
The query feature that was added to PageForms is yet quite limited in comparison to SFS. For example, the feature does not support printouts or any ask params like Worth to also note separately that the |
Well, for non-autocompletion input, like dropdowns, couldn't you just do Anyway, I think the most persuasive argument for adding SFS to Canasta is that it requires Composer for installation - which means that it would be difficult to add it separately, given the existing issues with Composer add-ons. Although, on that note - I am surprised to see that this current patch adds SFS via both Git and Composer. Are those both necessary? |
I don't think so, SFS allows using dependent fields values in a query runtime, which is not the case for
Actually, it's not both, the extension is added via Git, but it still needs |
Right, I know about the dynamic/dependent part of SFS - I think it's not that important, given that Page Forms has "values dependent on", but maybe it has some specific uses that "values dependent on" cannot handle. Anyway, I still think the key aspect is that SFS has (for no good reason) a Composer requirement - and given Canasta's incomplete support for Composer, that seems to make SFS inclusion in Canasta necessary. I'm still curious about the Composer/Git thing, though. What do you mean by "autoloading"? And what would happen if you just downloaded SFS via Git, then added a wfLoadExtension() call for it? |
I did not use For the "autoloading" I am referring to the |
Well, whatever benefits SFS provides (I still think they are minimal), the fact that it requires Composer for installation makes, in my opinion, a strong reason for including it. Hopefully Composer support will be fixed before the next major Canasta version, and then we can discuss getting rid of SFS (I didn't say get rid of it, I said discuss it), but for now I think it makes sense to include it. So this patch looks good to me. |
Adds https://www.mediawiki.org/wiki/Extension:Semantic_Forms_Select version
4.0.0-beta