Differences

This shows you the differences between two versions of the page.

--- tips:subscription [2013-08-27 17:48] – 212.87.131.201
+++ tips:subscription [2015-09-14 11:33] (current) – davemidd
@@ Line 1: / Line 1: @@
 Linked from [[:subscription]]
-FIXME This workaround does not work in current versions (Angua or newer)
 ^It appears this is now included in DokuWiki 2009-02-14 (which confused me after upgrading, cos I initially thought the upgrade had broken subscriptions! LOL ). It's hardcoded to not send subscription emails to the editor of the page. If for some reason you want the old behaviour back (i.e. //do// get emails for your own edits), modify the call to ''subscriber_addresslist'' in inc/common.php so that the second parameter is ''true'' (the logic is inverted compared to the old workaround below).^
@@ Line 9: / Line 7: @@
 You may wish to avoid sending emails to the author of the change, if the author is subscribed. Because- of course the author already knows that he/she made a change. I submit the below code change in order to work around this issue. It may be desirable to add a config flag for this setting. Possibly something each individual user can toggle for him/herself. At this point, it is on for everyone. ~Sherri W (www.start.ofitall.com).
+In Detritus (& possibly older?) a change is needed to 'function notify()' in inc/common.php, to set 'self' to 'true':
+<code>
+diff -Naur common.php.orig common.php
+--- common.php.orig
++++ common.php
+@@ -1318,7 +1318,7 @@
+     } elseif($who == 'subscribers') {
+         if(!actionOK('subscribe')) return false; //subscribers enabled?
+         if($conf['useacl'] && $INPUT->server->str('REMOTE_USER') && $minor) return false; //skip minors
+-        $data = array('id' => $id, 'addresslist' => '', 'self' => false, 'replacements' => $replace);
++        $data = array('id' => $id, 'addresslist' => '', 'self' => true, 'replacements' => $replace);
+         trigger_event(
+             'COMMON_NOTIFY_ADDRESSLIST', $data,
+             array(new Subscription(), 'notifyaddresses')
+</code>
+ --- [[user>davemidd|davemidd]] //2015-09-14 11:32//
 ===== Implementation Of Work-Around =====