Uniform Hooks

[adamkiss]

This is a work in progress of allowing to call hooks on various events; So far, I've added the hooks, and events of "error", "success" and "always", called in the done() function.

There are no tests yet, but I will add them if/when we iron out naming/existence of this feature and its parts.

[mzur]

Thanks and sorry for being silent for so long! I think this looks good, as it solves your use case from #238. But I don't see a need for the always hook. Couldn't this just be called next to the form execution?

[adamkiss]

@mzur No problem at all! This is open source after all! (and uniform isn't the only plugin I'm using patched from a custom repo 🦅 )

---

Since I based the naming originally on github, I just included the always hook as well. But looking at the order of execution now, if you have a standard "flash/redirect" form and a code like this:

$uniform = $uniform->guard()->guard()->action()->done();
doCustomStuff($uniform);

the doCustomStuff never gets called, because done() redirects the page at that point.

->on('always') on the other hand gets called (You also create a compact form definition code block, but that in itself prob wouldn't be a sufficient argument)

[adamkiss]

Update: interestingly, if you don't add the ->done() call (manually), the hooks don't get called in case of success, but the form works (and the doCustomStuff() gets called)

I don't know what to make of it.