The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.It seems that many people like the MLT feature and it seems a bit silly to have it as a separate module.
At the least, we could better re-use the existing blocks admin interface.
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | move-mlt-framework-453338-12.patch | 39.21 KB | pwolanin |
| #11 | move-mlt-framework-453338-11.patch | 21.84 KB | JacobSingh |
| #9 | move-mlt-framework-453338-9.patch | 38.32 KB | pwolanin |
| #8 | move-mlt-framework-453338-8.patch | 38.26 KB | pwolanin |
| #7 | simplify-mlt-453338-7.patch | 25.52 KB | pwolanin |
Comments
Comment #1
pwolanin commentedHere's a patch that simplifies things quite a bit - I'm a little unsure whether we should actually roll this into one of the other modules since that will make the checks on the module and delta attributes a little harder.
Comment #2
pwolanin commentedA little more, but somehow not working...
Comment #3
pwolanin commentedok, now working - jsut missed one variable conversion.
Comment #4
pwolanin commentedFixed a fatal error and clean up the form a little more.
Comment #5
robertdouglass commentedfrom scanning the patch I think I like this idea. Just to confirm - all MLT blocks will have the same level of configurability? Seems so from the patch.
Next step after this: decouple getting MLT recommendations from "blocks" altogether so that they can be used as a normal API.
Comment #6
pwolanin commented@robert - this does not change configurability at all - just relocates the configuration to the actual per-block configuration form.
Easy enough to take one more step to make the core MLT function more of an API than before.
Comment #7
pwolanin commentedComment #8
pwolanin commentedWith some minor hacking wrt the delta, we can just move this into the framework like so. How does this seem?
Comment #9
pwolanin commentedA couple more little fixes.
Comment #10
janusman commentedTested under latest 6.x-1.x-DEV
Comment #11
JacobSingh commentedWorks good.
I made one tiny change to make the message after being redirected a little more digestible.
It would be nice to include a "What's that?" link next to add new recommended content block when we have some docs.
Comment #12
pwolanin commentedcommitted with one more minor change - make the compaison field checkboxes required so that at least one must be selected.