[Lift] Re: Warnings for missing binds?
I'd like to see this - had a similar hour of headscratching myself this morning. I wouldn't be adverse to info logs on successful binds either. Derek Chen-Becker wrote: I just spent the better part of half a day trying to track down an issue with binding that ended up being a subtle typo. I'm thinking of adding in some logging (in dev mode) for the bind function to warn about unused bind elements and unused bind params, just to better help locate issues like this. Any objections? Derek --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Lift group. To post to this group, send email to liftweb@googlegroups.com To unsubscribe from this group, send email to liftweb+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/liftweb?hl=en -~--~~~~--~~--~--~--- begin:vcard fn:Matt Williams n:Williams;Matt org:Makeable Solutions Ltd adr;dom:;;113-116 Bute Street;Cardiff;;CF10 5EQ email;internet:m...@makeable.co.uk title:Chief Platform Architect tel;work:02920 007543 tel;cell:07958 208482 x-mozilla-html:FALSE url:http://www.makeable.co.uk version:2.1 end:vcard smime.p7s Description: S/MIME Cryptographic Signature
[Lift] Re: Warnings for missing binds?
That would be nice ... but still what was the problem with unused bind params? ... could you please elaborate a bit? Br's, Marius On Jun 26, 6:32 pm, Derek Chen-Becker dchenbec...@gmail.com wrote: I just spent the better part of half a day trying to track down an issue with binding that ended up being a subtle typo. I'm thinking of adding in some logging (in dev mode) for the bind function to warn about unused bind elements and unused bind params, just to better help locate issues like this. Any objections? Derek --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Lift group. To post to this group, send email to liftweb@googlegroups.com To unsubscribe from this group, send email to liftweb+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/liftweb?hl=en -~--~~~~--~~--~--~---
[Lift] Re: Warnings for missing binds?
+1 - Derek Chen-Beckerdchenbec...@gmail.com wrote: I just spent the better part of half a day trying to track down an issue with binding that ended up being a subtle typo. I'm thinking of adding in some logging (in dev mode) for the bind function to warn about unused bind elements and unused bind params, just to better help locate issues like this. Any objections? Derek --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Lift group. To post to this group, send email to liftweb@googlegroups.com To unsubscribe from this group, send email to liftweb+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/liftweb?hl=en -~--~~~~--~~--~--~---
[Lift] Re: Warnings for missing binds?
I meant that it could warn you if your bind tag looks like: lift:bind name=content / But you made a mistake with your surround tag: lift:surround at=contet / (notice the misspelling) I was thinking that you would get two warnings: 1. One that nothing had matched the content name 2. One that there was an unused contet surround Derek On Fri, Jun 26, 2009 at 9:52 AM, marius d. marius.dan...@gmail.com wrote: That would be nice ... but still what was the problem with unused bind params? ... could you please elaborate a bit? Br's, Marius On Jun 26, 6:32 pm, Derek Chen-Becker dchenbec...@gmail.com wrote: I just spent the better part of half a day trying to track down an issue with binding that ended up being a subtle typo. I'm thinking of adding in some logging (in dev mode) for the bind function to warn about unused bind elements and unused bind params, just to better help locate issues like this. Any objections? Derek --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Lift group. To post to this group, send email to liftweb@googlegroups.com To unsubscribe from this group, send email to liftweb+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/liftweb?hl=en -~--~~~~--~~--~--~---
[Lift] Re: Warnings for missing binds?
Nice proposal! I'm noodling on adding a bit of framework to widgets, I got a bit frustrated with a widget today... Aöso, would be cool to make them OSGI components to be able to add them without rebooting. But now I'm babbling again -- Viktor On Jun 26, 2009 7:11 PM, Derek Chen-Becker dchenbec...@gmail.com wrote: I meant that it could warn you if your bind tag looks like: lift:bind name=content / But you made a mistake with your surround tag: lift:surround at=contet / (notice the misspelling) I was thinking that you would get two warnings: 1. One that nothing had matched the content name 2. One that there was an unused contet surround Derek On Fri, Jun 26, 2009 at 9:52 AM, marius d. marius.dan...@gmail.com wrote: That would be ni... --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Lift group. To post to this group, send email to liftweb@googlegroups.com To unsubscribe from this group, send email to liftweb+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/liftweb?hl=en -~--~~~~--~~--~--~---
[Lift] Re: Warnings for missing binds?
Ah right ... by all means please add this. On Jun 26, 7:57 pm, Derek Chen-Becker dchenbec...@gmail.com wrote: I meant that it could warn you if your bind tag looks like: lift:bind name=content / But you made a mistake with your surround tag: lift:surround at=contet / (notice the misspelling) I was thinking that you would get two warnings: 1. One that nothing had matched the content name 2. One that there was an unused contet surround Derek On Fri, Jun 26, 2009 at 9:52 AM, marius d. marius.dan...@gmail.com wrote: That would be nice ... but still what was the problem with unused bind params? ... could you please elaborate a bit? Br's, Marius On Jun 26, 6:32 pm, Derek Chen-Becker dchenbec...@gmail.com wrote: I just spent the better part of half a day trying to track down an issue with binding that ended up being a subtle typo. I'm thinking of adding in some logging (in dev mode) for the bind function to warn about unused bind elements and unused bind params, just to better help locate issues like this. Any objections? Derek --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Lift group. To post to this group, send email to liftweb@googlegroups.com To unsubscribe from this group, send email to liftweb+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/liftweb?hl=en -~--~~~~--~~--~--~---
[Lift] Re: Warnings for missing binds?
OK, pushed to master. If you typo your bind IDs, you'll get something like: WARN - No binding values match the lift:bind name attribute: content WARN - Unused binding values for lift:bind: contet Derek On Fri, Jun 26, 2009 at 1:09 PM, marius d. marius.dan...@gmail.com wrote: Ah right ... by all means please add this. On Jun 26, 7:57 pm, Derek Chen-Becker dchenbec...@gmail.com wrote: I meant that it could warn you if your bind tag looks like: lift:bind name=content / But you made a mistake with your surround tag: lift:surround at=contet / (notice the misspelling) I was thinking that you would get two warnings: 1. One that nothing had matched the content name 2. One that there was an unused contet surround Derek On Fri, Jun 26, 2009 at 9:52 AM, marius d. marius.dan...@gmail.com wrote: That would be nice ... but still what was the problem with unused bind params? ... could you please elaborate a bit? Br's, Marius On Jun 26, 6:32 pm, Derek Chen-Becker dchenbec...@gmail.com wrote: I just spent the better part of half a day trying to track down an issue with binding that ended up being a subtle typo. I'm thinking of adding in some logging (in dev mode) for the bind function to warn about unused bind elements and unused bind params, just to better help locate issues like this. Any objections? Derek --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Lift group. To post to this group, send email to liftweb@googlegroups.com To unsubscribe from this group, send email to liftweb+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/liftweb?hl=en -~--~~~~--~~--~--~---
[Lift] Re: Warnings for missing binds?
Indentation still looks horked. I give up :( On Fri, Jun 26, 2009 at 3:04 PM, Derek Chen-Becker dchenbec...@gmail.comwrote: OK, just pushed again. Emacs did some weird things to indentation, so I fixed it. Derek On Fri, Jun 26, 2009 at 1:49 PM, Derek Chen-Becker dchenbec...@gmail.comwrote: OK, pushed to master. If you typo your bind IDs, you'll get something like: WARN - No binding values match the lift:bind name attribute: content WARN - Unused binding values for lift:bind: contet Derek On Fri, Jun 26, 2009 at 1:09 PM, marius d. marius.dan...@gmail.comwrote: Ah right ... by all means please add this. On Jun 26, 7:57 pm, Derek Chen-Becker dchenbec...@gmail.com wrote: I meant that it could warn you if your bind tag looks like: lift:bind name=content / But you made a mistake with your surround tag: lift:surround at=contet / (notice the misspelling) I was thinking that you would get two warnings: 1. One that nothing had matched the content name 2. One that there was an unused contet surround Derek On Fri, Jun 26, 2009 at 9:52 AM, marius d. marius.dan...@gmail.com wrote: That would be nice ... but still what was the problem with unused bind params? ... could you please elaborate a bit? Br's, Marius On Jun 26, 6:32 pm, Derek Chen-Becker dchenbec...@gmail.com wrote: I just spent the better part of half a day trying to track down an issue with binding that ended up being a subtle typo. I'm thinking of adding in some logging (in dev mode) for the bind function to warn about unused bind elements and unused bind params, just to better help locate issues like this. Any objections? Derek --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Lift group. To post to this group, send email to liftweb@googlegroups.com To unsubscribe from this group, send email to liftweb+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/liftweb?hl=en -~--~~~~--~~--~--~---