#foswiki 2017-01-20,Fri

↑back Search ←Prev date Next date→ Show only urls(Click on time to select a line by its url)

WhoWhatWhen
gac410vrurg: For Item13892, Would you agree that we could "fix" this by documenting that CALCULATE should be the preferred solution? [01:42]
FoswikiBothttps://foswiki.org/Tasks/Item13892 [ Item13892: CALC doesn't work in %SEARCH despite of what documention says. ] [01:42]
vrurgProbably yes. I can't remember details of the problem for now. But as long as CALCULATE fixed our problem I think it's ok. [01:46]
gac410Okay. I'll document that CALCULATE is the preferred macro to use in searches, along with the restriction of expansion order. Thanks. [01:47]
vrurgyw
I'm anyway is so deep into low-level code that would have to relearn most of the macros from scratch. ;) Didn't create a complicated page for months now.
[01:47]
gac410:) [01:49]
.......... (idle for 48mn)
GithubBot[distro] gac410 pushed 1 new commit to Release02x01: https://git.io/vMSZG
distro/Release02x01 89425d9 George Clark: Item13892: Clarify docs of CALC vs CALCULATE
[02:37]
***GithubBot has left [02:37]
FoswikiBothttps://foswiki.org/Tasks/Item13892 [ Item13892: CALC doesn't work in %SEARCH despite of what documention says. ] [02:37]
GithubBot[distro] gac410 pushed 1 new commit to master: https://git.io/vMSZc
distro/master addab09 George Clark: Item13892: Merge branch 'Release02x01'
[02:37]
***GithubBot has left [02:37]
.......... (idle for 46mn)
SvenDowideit_ has quit IRC (Read error: Connection reset by peer) [03:23]
.......................... (idle for 2h8mn)
gac410 has left [05:31]
............................ (idle for 2h19mn)
ChanServ sets mode: +o cdot [07:50]
..................................... (idle for 3h1mn)
GithubBot[WorkflowPlugin] cdot pushed 1 new commit to master: https://git.io/vMS7W
WorkflowPlugin/master 22ae343 cdot: Item14210: allow override of reminder template by a local topic template. Run as admin user to overcome permission limitations (OK for a cronned task). Add some doc.
[10:51]
***GithubBot has left [10:51]
FoswikiBothttps://foswiki.org/Tasks/Item14210 [ Item14210: Support re-notification when a topic is stuck in a state ] [10:51]
GithubBot[WorkflowPlugin] cdot pushed 1 new commit to master: https://git.io/vMS71
WorkflowPlugin/master 3e12243 cdot: Item14210: allow override of reminder template by a local topic template. Run as admin user to overcome permission limitations (OK for a cronned task). Add some doc.
[10:53]
***GithubBot has left [10:53]
GithubBot[WorkflowPlugin] cdot pushed 1 new commit to master: https://git.io/vMS5B
WorkflowPlugin/master b5af79b cdot: Item14048: long overdue merge
[10:58]
***GithubBot has left [10:58]
FoswikiBothttps://foswiki.org/Tasks/Item14048 [ Item14048: multiple critical improvements to WorkflowPlugin ] [10:58]
..................................... (idle for 3h1mn)
***ChanServ sets mode: +o Lynnwood [13:59]
......... (idle for 40mn)
ChanServ sets mode: +o gac410 [14:39]
..... (idle for 21mn)
Lynnwoodgac410 - good morning. A bit of an update on ImmediateNotifyPlugin. I installed the copy from github and see that it is quite a bit more developoed than the published version in Testing web.
For starters, the Configure setting interface is defined.
[15:00]
gac410Cool. After 4+ years, I've forgotten what I had done. [15:01]
LynnwoodThere was one little hiccup. I enabled SMTP notification (which was not enabled by default) and set IMMEDIATENOTIFY in a topic. When I saved the topic, I got an error. I'll give you the error from apache log....
Can't call method "disconnect" without a package or object reference at /var/www/foswiki/lib/Foswiki/Plugins/Immedia teNotifyPlugin.pm line 114.,
[15:03]
***gac410 sets mode: +o FoswikiBot [15:04]
gac410oops wrong switch [15:04]
***gac410 sets mode: -o FoswikiBot
gac410 sets mode: +v FoswikiBot
[15:04]
LynnwoodI took a look at that line and since it really didn't look like it would apply to SMTP, I simply commented it out.
And it works without complaint now.
I will post a task about it.
[15:05]
gac410Y. That is only for the handlers that establish connections. Needs an exception for SMTP [15:06]
LynnwoodI see in the configure that you were at least in the process of adding a couple other notification types. [15:06]
gac410This would probably work:        next unless ( $handler && $handler->can('disconnect') );
Yeah. I really wanted to fold in tweet, ... There are several old twiki extensions that all did similar things, and having one caller made sense.
irc notify too.
[15:07]
Lynnwoodyea... it was irc that i was interested in (potentially). Actually i must of seen that in the code. it's not in configure. [15:08]
gac410IIRC all I did was start to create the skeleton handlers ... no real work on them. [15:09]
Lynnwoodyep
For now, i'm just delighted that I've been able to get it running for SMTP
[15:09]
gac410Probably the perl modules for the handlers all need review and updating. Things change in cpan land, over 4 years
Tweet and IRC would be nice - for ex to announce extensions updates. But would need some filtering, no sense tweeting for all the uploads when an extension is released.
[15:10]
Lynnwood...which none are necessary for SMTP. I didn't even install them. [15:12]
gac410right. SMTP uses the Foswiki::Net tools - thankfully. That really simplifies it [15:13]
FoswikiBothttps://trunk.foswiki.org/System/PerlDoc?module=Foswiki::Net [15:13]
.... (idle for 16mn)
Lynnwoodgac410 - just for the heck of it, i tried inserting the code you suggested above in the plugin. Afraid it still didn't work.
...doen't really matter to me for now cause i can make it work by commenting out line 114.
If i understood correctly, i used that code on line 113.
Ah, i see there had been another error in apache log:
Use of uninitialized value $methodString in concatenation (.) or string at /var/www/foswiki/lib/Foswiki/Plugins/ImmediateNotifyPlugin.pm line 208.
[15:29]
gac410Hm ... Okay. At some point I'll work on it a bit again.
That one is down in processName - It seems to be unrelated to the finish() call
I guess there was a reason that I didn't release it ;)
[15:34]
LynnwoodWell, at least for SMTP, I think the fix needed would be minimal.
https://foswiki.org/Tasks/Item14284
[15:37]
gac410Okay thanks Lynnwood ... I guess you are going to twist my arm until I do something here :D [15:38]
Lynnwoodyou've already been more than helpful on my end. At least my user will be satisfied for now. [15:39]
................ (idle for 1h15mn)
GithubBot[distro] MichaelDaum pushed 1 new commit to Release02x01: https://git.io/vM9K1
distro/Release02x01 9f711fa MichaelDaum: Item14285: fixed error clicking on disabled btn
[16:54]
***GithubBot has left [16:54]
FoswikiBothttps://foswiki.org/Tasks/Item14285 [ Item14285: don't try to action upon disabled toobar buttons ] [16:54]
gac410MichaelDaum ... if you are lurking somewhere, Should I merge Item14281 changes - You commented you checked the code but didn't indicate if it looked okay. ;) [16:58]
FoswikiBothttps://foswiki.org/Tasks/Item14281 [16:58]
.... (idle for 16mn)
GithubBot[distro] FoswikiBot pushed 3 new commits to master: https://git.io/vM9Pm
distro/master 3fe2ebc Raul F Rodriguez: Item13884: Translations updated using Weblate (French)...
distro/master adfd38c Raul F Rodriguez: Item13884: Translations updated using Weblate (French)...
distro/master 9d29f97 Guido Brugnara: Item13884: Translations updated using Weblate (Italian)...
[17:14]
***GithubBot has left [17:14]
FoswikiBothttps://foswiki.org/Tasks/Item13884 [ Item13884: Translation changes for master and Foswiki 2.1 ] [17:14]
GithubBot[distro] FoswikiBot pushed 3 new commits to Release02x01: https://git.io/vM9Pc
distro/Release02x01 30256b1 Guido Brugnara: Item13884: Translated using Weblate (Italian)...
distro/Release02x01 07617f3 Raul F Rodriguez: Item13884: Translated using Weblate (French)...
distro/Release02x01 065a863 Guido Brugnara: Item13884: Translated using Weblate (Italian)...
[17:15]
***GithubBot has left [17:15]
GithubBot[distro] gac410 pushed 1 new commit to Release02x01: https://git.io/vM9PV
distro/Release02x01 335d48d George Clark: Item13884: xgettext run, no string changes
[17:17]
***GithubBot has left [17:17]
GithubBot[distro] gac410 pushed 1 new commit to master: https://git.io/vM9PP
distro/master e6658c2 George Clark: Merge branch 'Release02x01'
[17:18]
***GithubBot has left [17:18]
GithubBot[distro] gac410 pushed 1 new commit to master: https://git.io/vM9Ph
distro/master b1227d6 George Clark: Item13884: master xgettext run, no string changes
[17:21]
***GithubBot has left [17:21]

↑back Search ←Prev date Next date→ Show only urls(Click on time to select a line by its url)