review.el 69.3 KB
Newer Older
David Byers's avatar
David Byers committed
1
;;;;; -*-coding: iso-8859-1;-*-
Linus Tolke's avatar
Linus Tolke committed
2
3
;;;;;
;;;;; $Id$
4
;;;;; Copyright (C) 1991-2002  Lysator Academic Computer Association.
Linus Tolke's avatar
Linus Tolke committed
5
;;;;;
6
;;;;; This file is part of the LysKOM Emacs LISP client.
Linus Tolke's avatar
Linus Tolke committed
7
8
9
;;;;; 
;;;;; LysKOM is free software; you can redistribute it and/or modify it
;;;;; under the terms of the GNU General Public License as published by 
10
;;;;; the Free Software Foundation; either version 2, or (at your option) 
Linus Tolke's avatar
Linus Tolke committed
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
;;;;; any later version.
;;;;; 
;;;;; LysKOM is distributed in the hope that it will be useful, but WITHOUT
;;;;; ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
;;;;; FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
;;;;; for more details.
;;;;; 
;;;;; You should have received a copy of the GNU General Public License
;;;;; along with LysKOM; see the file COPYING.  If not, write to
;;;;; Lysator, c/o ISY, Linkoping University, S-581 83 Linkoping, SWEDEN,
;;;;; or the Free Software Foundation, Inc., 675 Mass Ave, Cambridge, 
;;;;; MA 02139, USA.
;;;;;
;;;;; Please mail bug reports to bug-lyskom@lysator.liu.se. 
;;;;;
Per Cederqvist's avatar
.    
Per Cederqvist committed
26
27
28
29
30
31
32
33
34
35
36
37
38
;;;; ================================================================
;;;; ================================================================
;;;;
;;;; File: review.el
;;;;
;;;; This file contains functions to review articles in different ways.
;;;; Both the review commands themselves, the functions called by them
;;;; and the underlying functions are here.
;;;;
;;;; Most, if not all, of these functions are written by Linus Tolke.
;;;;


39
40
41
42
(setq lyskom-clientversion-long 
      (concat lyskom-clientversion-long
	      "$Id$\n"))

43
44
45
(eval-when-compile
  (require 'lyskom-command "command"))

David Byers's avatar
David Byers committed
46
47
48
49
(put 'lyskom-cant-review-error 'error-conditions '(error lyskom-error lyskom-review-error))
(put 'lyskom-cant-review-error 'error-message "Can't review")
(put 'lyskom-review-error 'error-condiions '(error lyskom-error))
(put 'lyskom-review-error 'error-message "Review error")
50

51

David Byers's avatar
David Byers committed
52
53
54
;;; ======================================================================
;;; State-saving variables
;;;
55
;;; All these need to be buffer-local
David Byers's avatar
David Byers committed
56

57
58
59
60
61
62
63
64
65
66
67
68
(def-kom-var lyskom-last-review-by nil local)
(def-kom-var lyskom-last-review-to nil local)
(def-kom-var lyskom-last-review-num nil local)
(def-kom-var lyskom-last-review-pmark nil local)
(def-kom-var lyskom-last-review-cmark nil local)
(def-kom-var lyskom-last-review-saved-result-list nil local)
(def-kom-var lyskom-last-review-saved-by-list nil local)
(def-kom-var lyskom-last-review-saved-to-list nil local)
(def-kom-var lyskom-last-review-saved-result-size 0 local)
(def-kom-var lyskom-last-review-saved-smallest nil local)
(def-kom-var lyskom-last-review-saved-largest nil local)
(def-kom-var lyskom-have-review nil local)
David Byers's avatar
David Byers committed
69

70
71
72
73
74
75
76
77
78
79
80
81
(def-kom-var lyskom-last-unread-by nil local)
(def-kom-var lyskom-last-unread-to nil local)
(def-kom-var lyskom-last-unread-num nil local)
(def-kom-var lyskom-last-unread-pmark nil local)
(def-kom-var lyskom-last-unread-cmark nil local)
(def-kom-var lyskom-last-unread-saved-result-list nil local)
(def-kom-var lyskom-last-unread-saved-by-list nil local)
(def-kom-var lyskom-last-unread-saved-to-list nil local)
(def-kom-var lyskom-last-unread-saved-result-size 0 local)
(def-kom-var lyskom-last-unread-saved-smallest nil local)
(def-kom-var lyskom-last-unread-saved-largest nil local)
(def-kom-var lyskom-have-unread nil local)
David Byers's avatar
David Byers committed
82

David Byers's avatar
David Byers committed
83

84
85
86
87
88
(defun lyskom-remove-zeroes (a)
  "Returns a copy of list where all zeroes are removed."
  (delq 0 (copy-sequence a)))


David Byers's avatar
David Byers committed
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
;;; ================================================================
;;; Temoporary change of mark-as-read

(defun kom-toggle-mark-as-read-prefix ()
  "Read one key sequence and run one command with state of
kom-review-marks-texts-as-read toggled."
  (interactive)
  (let* ((kom-review-marks-texts-as-read (not kom-review-marks-texts-as-read))
         (sequence (read-key-sequence
                    (format "%s: " 
                            (lyskom-get-string
                             (if kom-review-marks-texts-as-read 
                                 'review-marking-as-read
                               'review-not-marking-as-read)))))
         (command (lookup-key (current-local-map) sequence)))
    (when (commandp command)
      (call-interactively command))))

107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
(defun kom-toggle-cache-prefix ()
  "Read one key sequence and run one command with state of
kom-review-marks-texts-as-read toggled."
  (interactive)
  (let* ((kom-review-uses-cache (not kom-review-uses-cache))
         (sequence (read-key-sequence
                    (format "%s: " 
                            (lyskom-get-string
                             (if kom-review-uses-cache 
                                 'review-using-cache
                               'review-not-using-cache)))))
         (command (lookup-key (current-local-map) sequence)))
    (when (commandp command)
      (call-interactively command))))

David Byers's avatar
David Byers committed
122

Per Cederqvist's avatar
.    
Per Cederqvist committed
123
124
125
;;; ================================================================
;;;              ]terse av, till - Review by X to Conference Y.

David Byers's avatar
David Byers committed
126
;;; Author: David Byers
127

Per Cederqvist's avatar
.    
Per Cederqvist committed
128

129
(def-kom-command kom-review-all ()
130
  "Review all texts written by a particular author to a particular
131
132
133
134
conference. This can also be accomplished by using `kom-review-by-to'
and specifying zero texts.

See `kom-review-uses-cache', `kom-review-priority' and
135
`kom-review-marks-texts-as-read' for information on settings that affect
136
all review-related functions."
137
138
139
140
  (interactive)
  (lyskom-tell-internat 'kom-tell-review)
  (lyskom-review-by-to 0))

David Byers's avatar
David Byers committed
141
(def-kom-command kom-unread-all ()
142
  "Mark all texts written by a particular author to a particular
David Byers's avatar
David Byers committed
143
144
145
146
147
conference as unread. This can also be accomplished by using
`kom-unread-by-to' and specifying zero texts."
  (interactive)
  (lyskom-unread-by-to 0))

Per Cederqvist's avatar
.    
Per Cederqvist committed
148

149
(def-kom-command kom-review-more (count)
150
  "Review more texts using the same critera as the last review
151
152
153
154
performed with `kom-review-by-to'. The review will be resumed where
the previous review finished.

See `kom-review-uses-cache', `kom-review-priority' and
155
`kom-review-marks-texts-as-read' for information on settings that affect
156
all review-related functions."
Ulrik Haugen's avatar
Ulrik Haugen committed
157
  (interactive "P")
David Byers's avatar
David Byers committed
158
159
  (if (not lyskom-have-review)
      (lyskom-format-insert 'no-review-done)
160
161
162
163
    (let* ((count (or count
		      (lyskom-read-number
		       (lyskom-get-string 'review-how-many-more)
		       (abs lyskom-last-review-num))))
David Byers's avatar
David Byers committed
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
           (info (progn (if (and (listp count)
                                 (integerp (car count))
                                 (null (cdr count)))
                            (setq count (car count)))
                        (cond ((zerop count) 
                               (setq count nil)
                               (lyskom-get-string 'review-rest))
                              ((> count 0)
                               (lyskom-format (lyskom-get-string 'review-more)
                                              count)))))
           (by lyskom-last-review-by)
           (to lyskom-last-review-to))

      (lyskom-format-insert 'review-more-info-by-to
                            info
                            (if (zerop by)
                                (lyskom-get-string 'anybody)
                              by)
                            (if (zerop to)
                                (lyskom-get-string 'all-confs)
                              to))
    
      (condition-case arg
          (let ((list (lyskom-get-texts-by-to by to count t)))
            (setq lyskom-last-review-num 
                  (if (< lyskom-last-review-num 0)
                      (- count)
                    count))
            (if list
193
194
195
196
197
198
199
                (lyskom-review-enter-read-info
                 (lyskom-create-read-info
                  'REVIEW
                  nil
                  (lyskom-review-get-priority)
                  (lyskom-create-text-list list)
                  nil t) t)
David Byers's avatar
David Byers committed
200
201
202
203
204
              (lyskom-insert-string 'no-such-text)))
        (lyskom-review-error (if arg
                                 nil
                               (lyskom-insert-string 'no-such-text)))))))

David Byers's avatar
David Byers committed
205
(def-kom-command kom-unread-more (count)
206
  "Mark more texts unread using the same critera as the last 
David Byers's avatar
David Byers committed
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
mark unread performed with `kom-unread-by-to'."
  (interactive "P")
  (if (not lyskom-have-unread)
      (lyskom-format-insert 'no-unread-done)
    (let* ((count (or count
		      (lyskom-read-number
		       (lyskom-get-string 'unread-how-many-more)
		       (abs lyskom-last-unread-num))))
           (info (progn (if (and (listp count)
                                 (integerp (car count))
                                 (null (cdr count)))
                            (setq count (car count)))
                        (cond ((zerop count) 
                               (setq count nil)
                               (lyskom-get-string 'unread-rest))
                              ((> count 0)
                               (lyskom-format (lyskom-get-string 'unread-more)
                                              count)))))
           (by lyskom-last-unread-by)
           (to lyskom-last-unread-to))

      (lyskom-format-insert 'unread-more-info-by-to
                            info
                            (if (zerop by)
                                (lyskom-get-string 'anybody)
                              by)
                            (if (zerop to)
                                (lyskom-get-string 'all-confs)
                              to))
    
      (condition-case arg
238
          (let ((list (lyskom-get-texts-by-to by to count t t)))
David Byers's avatar
David Byers committed
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
            (setq lyskom-last-unread-num 
                  (if (< lyskom-last-unread-num 0)
                      (- count)
                    count))
            (if list
              (lyskom-traverse text-no list
                (unless (lyskom-mark-unread text-no)
                  (lyskom-format-insert 'cant-mark-text-unread
                                        text-no
                                        (lyskom-get-error-text lyskom-errno))))
              (lyskom-insert-string 'no-such-text)))
        (lyskom-review-error (if arg
                                 nil
                               (lyskom-insert-string 'no-such-text)))))))

254

David Byers's avatar
David Byers committed
255

256
(def-kom-command kom-review-first (&optional count)
257
  "Review the first N texts written by a particular author to some
258
259
260
261
262
263
264
conference. With no author specified, review texts by all authors.
With zero texts specified, review all text. With no conference
specified, review texts to all conferences. With a negative number of
texts, review the last N texts instead of the first (you can use
`kom-review-by-to' instead.

See `kom-review-uses-cache', `kom-review-priority' and
265
`kom-review-marks-texts-as-read' for information on settings that affect
266
all review-related functions."
267
  (interactive "P")
268
269
270
271
272
  (lyskom-tell-internat 'kom-tell-review)
  (lyskom-review-by-to (- (or count
                           (lyskom-read-number
                            (lyskom-get-string 'review-how-many) 1)))))

David Byers's avatar
David Byers committed
273
(def-kom-command kom-unread-first (&optional count)
274
  "Mark the first N texts written by a particular author to some
David Byers's avatar
David Byers committed
275
276
277
278
279
280
281
282
283
284
conference as unread. With no author specified, review texts by all 
authors. With zero texts specified, review all text. With no conference
specified, review texts to all conferences. With a negative number of
texts, review the last N texts instead of the first (you can use
`kom-unread-by-to' instead."
  (interactive "P")
  (lyskom-unread-by-to (- (or count
                           (lyskom-read-number
                            (lyskom-get-string 'unread-how-many) 1)))))

285

David Kågedal's avatar
David Kågedal committed
286
(def-kom-command kom-review-by-to (&optional count)
287
  "Review the last N texts written by a particular author to some
288
289
290
291
292
293
294
conference. With no author specified, review texts by all authors.
With zero texts specified, review all text. With no conference
specified, review texts to all conferences. With a negative number of
texts, review the last N texts instead of the first (you can use
`kom-review-first' instead.

See `kom-review-uses-cache', `kom-review-priority' and
295
`kom-review-marks-texts-as-read' for information on settings that affect
296
all review-related functions."
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
  (interactive "P")
  (lyskom-review-by-to (or count
                           (lyskom-read-number
                            (lyskom-get-string 'review-how-many) 1))))
  


(defun lyskom-review-by-to (count)
  "Common function for kom-review-by-to and kom-review-first"
  (let* ((info (progn (if (and (listp count)
                               (integerp (car count))
                               (null (cdr count)))
                          (setq count (car count)))
                      (cond ((zerop count) 
                             (setq count nil)
312
                             (lyskom-get-string 'everything))
313
314
315
316
317
                            ((> count 0)
                             (lyskom-format 'latest-n count))
                            ((< count 0)
                             (lyskom-format 'first-n
                                            (- count))))))
318
319
320
321
         (by (lyskom-read-conf-no `(review-by-whom ,info)
                                  '(pers) t nil t))
         (to (lyskom-read-conf-no `(review-to-conf ,info)
                                  '(all) t nil t)))
David Kågedal's avatar
David Kågedal committed
322
323

    (if (not (zerop to))
324
        (cache-del-conf-stat to))
David Kågedal's avatar
David Kågedal committed
325
    (if (not (zerop by)) 
326
        (cache-del-pers-stat by))
327

David Byers's avatar
David Byers committed
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
    (lyskom-format-insert 'review-info-by-to
                          info
                          (if (zerop by)
                              (lyskom-get-string 'anybody)
                            by)
                          (if (zerop to)
                              (lyskom-get-string 'all-confs)
                            to))

    (setq lyskom-last-review-by by)
    (setq lyskom-last-review-to to)
    (setq lyskom-last-review-num count)
    (setq lyskom-last-review-pmark nil)
    (setq lyskom-last-review-cmark nil)
    (setq lyskom-last-review-saved-result-list nil)
    (setq lyskom-last-review-saved-by-list nil)
    (setq lyskom-last-review-saved-to-list nil)
    (setq lyskom-last-review-saved-result-size 0)
    (setq lyskom-last-review-saved-smallest nil)
    (setq lyskom-last-review-saved-largest nil)
    (setq lyskom-have-review t)
David Kågedal's avatar
David Kågedal committed
349

350
351
352
    (condition-case arg
        (let ((list (lyskom-get-texts-by-to by to count)))
          (if list
353
354
355
356
357
358
359
              (lyskom-review-enter-read-info
               (lyskom-create-read-info
                'REVIEW
                nil
                (lyskom-review-get-priority)
                (lyskom-create-text-list list)
                nil t) t)
360
361
362
363
            (lyskom-insert-string 'no-such-text)))
      (lyskom-review-error (if arg
                               nil 
                             (lyskom-insert-string 'no-such-text))))))
364

David Kågedal's avatar
David Kågedal committed
365

David Byers's avatar
David Byers committed
366
(def-kom-command kom-unread-by-to (&optional count)
367
  "Mark the last N texts written by a particular author to some
David Byers's avatar
David Byers committed
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
conference as unread. With no author specified, review texts by all 
authors. With zero texts specified, review all text. With no conference
specified, review texts to all conferences. With a negative number of
texts, review the last N texts instead of the first (you can use
`kom-unread-first' instead."
  (interactive "P")
  (lyskom-unread-by-to (or count
                           (lyskom-read-number
                            (lyskom-get-string 'unread-how-many) 1))))
  

(defun lyskom-unread-by-to (count)
  "Common function for kom-review-by-to and kom-review-first"
  (let* ((info (progn (if (and (listp count)
                               (integerp (car count))
                               (null (cdr count)))
                          (setq count (car count)))
                      (cond ((zerop count) 
                             (setq count nil)
                             (lyskom-get-string 'everything))
                            ((> count 0)
                             (lyskom-format 'latest-n count))
                            ((< count 0)
                             (lyskom-format 'first-n
                                            (- count))))))
393
394
395
396
         (by (lyskom-read-conf-no `(unread-by-whom ,info)
                                  '(pers) t nil t))
         (to (lyskom-read-conf-no `(unread-to-conf ,info)
                                  '(all) t nil t)))
David Byers's avatar
David Byers committed
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425

    (if (not (zerop to))
        (cache-del-conf-stat to))
    (if (not (zerop by)) 
        (cache-del-pers-stat by))

    (lyskom-format-insert 'unread-info-by-to
                          info
                          (if (zerop by)
                              (lyskom-get-string 'anybody)
                            by)
                          (if (zerop to)
                              (lyskom-get-string 'all-confs)
                            to))

    (setq lyskom-last-unread-by by)
    (setq lyskom-last-unread-to to)
    (setq lyskom-last-unread-num count)
    (setq lyskom-last-unread-pmark nil)
    (setq lyskom-last-unread-cmark nil)
    (setq lyskom-last-unread-saved-result-list nil)
    (setq lyskom-last-unread-saved-by-list nil)
    (setq lyskom-last-unread-saved-to-list nil)
    (setq lyskom-last-unread-saved-result-size 0)
    (setq lyskom-last-unread-saved-smallest nil)
    (setq lyskom-last-unread-saved-largest nil)
    (setq lyskom-have-unread t)

    (condition-case arg
426
        (let ((list (lyskom-get-texts-by-to by to count nil t)))
David Byers's avatar
David Byers committed
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
          (if list
              (lyskom-traverse text-no list
                (unless (lyskom-mark-unread text-no)
                  (lyskom-format-insert 'cant-mark-text-unread
                                        text-no
                                        (lyskom-get-error-text lyskom-errno))))
            (lyskom-insert-string 'no-such-text)))
      (lyskom-review-error (if arg
                               nil 
                             (lyskom-insert-string 'no-such-text))))))






443
444
445
446
447
448
449
450
451
452
;;; ================================================================
;;; lyskom-get-texts-by-to
;;; Author: David Byers
;;;
;;; Call lyskom-get-texts-by, lyskom-get-texts-to or 
;;; lyskom-get-texts-by-and-to to get NUM texts by person 
;;; BY to conference TO.
;;;


453
(defun lyskom-get-texts-by-to (by to num &optional again do-unread)
David Byers's avatar
David Byers committed
454
  "Get NUM texts written by person number BY in conference number TO
455
Args: BY TO NUM AGAIN DO-UNREAD"
456
  (cond ((and (zerop by) 
457
              (zerop to)) (lyskom-get-texts-globally num again do-unread))
David Byers's avatar
David Byers committed
458
459
        ((zerop to) (lyskom-get-texts-by by num again))
        ((zerop by) (lyskom-get-texts-to to num again))
David Byers's avatar
David Byers committed
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
        ((and (eq by lyskom-pers-no)
              (not (eq to by))
              (let ((conf (blocking-do 'get-conf-stat to)))
                (and (conf-type->letterbox (conf-stat->conf-type conf))
                     (null (map->text-nos 
                            (blocking-do 'get-map
                                         (conf-stat->conf-no conf)
                                         (conf-stat->first-local-no conf)
                                         1))))))
         (lyskom-get-texts-by-generic 
          by num 
          (function 
           (lambda (x to)
             (let ((found nil))
               (lyskom-traverse misc (text-stat->misc-info-list x)
                 (setq found 
                       (or found
477
478
                           (and (memq (misc-info->type misc)
					lyskom-recpt-types-list)
David Byers's avatar
David Byers committed
479
480
481
482
483
                                (eq (misc-info->recipient-no misc) to)))))
               found)))
          (list to)
          again))
         (t (lyskom-get-texts-by-and-to by to num again))))
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519


;;; ============================================================
;;; lyskom-check-review-access
;;; Author: David Byers
;;;
;;; Check that we can access the conference map. If we can't some
;;; review functions are just not much fun
;;;

(defun lyskom-check-review-access (conf pers)
  "Check that we can review texts to CONF by PERS.
CONF is a conf-stat or t if we know we can access that conference.
PERS is a pers-stat or t if we know we can access that person.
This function signals an error if review is impossible"
  (cond ((null conf) 
         (lyskom-format-insert 'review-conf-gone)
         (signal 'lyskom-cant-review-error t))
        ((null pers)
         (lyskom-format-insert 'review-pers-gone)
         (signal 'lyskom-cant-review-error t))
        ((lyskom-conf-stat-p conf)
         (cond ((= 0 (conf-stat->no-of-texts conf))
                (lyskom-format-insert 'review-cant-read-empty)
                (signal 'lyskom-cant-review-error t))
               ((null (map->text-nos 
                       (blocking-do 'get-map
                                    (conf-stat->conf-no conf)
                                    (conf-stat->first-local-no conf)
                                    1)))
                (if (conf-type->letterbox
                     (conf-stat->conf-type conf))
                    (lyskom-format-insert 'review-cant-read-letterbox)
                  (lyskom-format-insert 'review-cant-read-conf))
                (signal 'lyskom-cant-review-error t))))))

520
521


522
523
524
525
526
;;; ============================================================
;;; lyskom-get-texts-globally
;;; Author: Per Cederquist, David Byers
;;;

527
(defun lyskom-get-texts-globally (num &optional again do-unread)
David Byers's avatar
David Byers committed
528
529
530
531
532
533
534
  "Get the last NUM texts created in LysKOM. If AGAIN is non-nil, keep
going from where we were before."
  (cond ((and again (null num))
         (setq num lyskom-last-review-num))
        ((and again (< lyskom-last-review-num 0)) 
         (setq num (- num)))
        ((null num) 
535
536
         (lyskom-format-insert
          (if do-unread 'cant-unread-everything 'cant-review-everything))
David Byers's avatar
David Byers committed
537
538
         (signal 'lyskom-cant-review-error t)))

539
  (let ((result nil)
David Byers's avatar
David Byers committed
540
541
542
        (textno (cond (again lyskom-last-review-cmark)
                      ((< num 0) 1)
                      (t (lyskom-maxint))))
543
544
545
546
547
        (op (if (< num 0)
                'find-next-text-no
              'find-previous-text-no)))

    (while (and (not (eq textno 0))
David Byers's avatar
David Byers committed
548
                (not (null textno))
549
550
                (< (length result) (abs num)))
      (setq textno (blocking-do op textno))
David Byers's avatar
David Byers committed
551
552
553
      (if textno
          (setq result (cons textno result))))
    (setq lyskom-last-review-cmark textno)
554
555
556
557
    (if (< num 0)
        (nreverse result)
      result)))

558

559
560
561
562
563
564
565
566
567
568
569
570
;;; ================================================================
;;; lyskom-get-letters-to
;;; Author: David Byers
;;; 
;;; Get letters by self that have a specified letterbox as
;;; recipient. Do this by linearly searching selfs letterbox
;;; map and look as every single doggone text-stat. What a drag.
;;;
;;; +++ FIXME: This is just get-texts-by right now. Need to filter stuff.
;;;

(defun lyskom-get-letters-to (persno recipient num &optional again pstart)
David Byers's avatar
David Byers committed
571
572
573
  "Get NUM texts written by PERSNO. Args: persno num

Cannot be called from a callback."
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
  (let ((persstat (blocking-do 'get-pers-stat persno)))
    (lyskom-check-review-access t persstat)

    (cond ((and again (null num)) (setq num lyskom-last-review-num))
          ((and again (< lyskom-last-review-num 0)) (setq num (- num))))

    (let* ((plow (or pstart (pers-stat->first-created-text persstat)))
           (phigh (1- (+ plow (pers-stat->no-of-created-texts persstat))))
           (result (if again
                       lyskom-last-review-saved-result-list
                     nil))
           (increment (if num (abs num)))
           (mark (cond (again lyskom-last-review-pmark)
                        ((and num (< num 0)) plow)
                        (t phigh)))
           (collector nil)
           (found nil)
           (start nil)
           (data nil))
    

      (if (null num)
          (setq num (1+ phigh)
                mark phigh
                increment (1+ phigh)))

      (while (and (<= mark phigh)
                  (>= mark plow)
                  (> (abs num) (length result)))

        (setq increment (min lyskom-fetch-map-nos increment))
        (setq start (if (< num 0)
                        mark
                      (- mark (1- increment))))
        (if (< start 0)
            (progn
              (setq increment (- increment start))
              (setq start 0)))
      
        (setq data (lyskom-remove-zeroes
                    (listify-vector
                     (map->text-nos
                      (blocking-do 'get-created-texts
                                   persno
                                   start
                                   increment)))))

        (setq collector (make-collector))
        (mapcar
         (function
          (lambda (x)
            (initiate-get-text-stat 
             'main 
             (function
              (lambda (x collector pers-no)
                (if (and x
                         (lyskom-is-recipient x pers-no))
                    (collector-push (text-stat->text-no x) collector))))
             x
             collector
             recipient)))
         data)

        (lyskom-wait-queue 'main)
        (setq found (nreverse (collector->value collector)))

        (if (> num 0)
            (setq result (nconc found result)
                  mark (- mark increment))
          (setq result (nconc result found)
                mark (+ mark increment)))
        (if (null found)
            (setq increment (min lyskom-fetch-map-nos (* increment 2)))
          (setq increment (- (abs num) (length result)))))

      (setq lyskom-last-review-pmark mark)

      (if (> num 0)
          (progn
            (setq lyskom-last-review-saved-result-list 
                  (nfirst (- (length result) num) result))
655
            (nthcdr (max 0 (- (length result) num)) result))
656
657
658
659
660
661
662
663
664
665
        (progn
          (setq lyskom-last-review-saved-result-list
                (nthcdr (- num) result))
          (nfirst (- num)  result))))))






666
667
;;; ================================================================
;;; lyskom-get-texts-by-and-to
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
695
;;; Author: David Byers
;;;
;;; Note: We can't assume that the conference's map of texts is
;;; sorted. If we could, it would be possible to simplify this
;;; function considerably without making it slower. 
;;;
;;; Problem: Construct part of the intersection between the user's
;;; created texts (the by-list) and the texts in a conference (the
;;; to-list) without bogging down the client, server or network and
;;; do it quickly!
;;;
;;; Idea: Construct the intersection incrementally without doing more
;;; work comparing things than we would if we had the full maps to
;;; start with.
;;;
;;; Solution: Get one segment of the by-list (call the nth segment
;;; by_n) and to-list (call the nth segment to_n) at a time until we
;;; are done or until both are exhausted. 
;;;
;;; In each iteration do the following: Calculate the intersection
;;; between to_n with each of the previous by_i leaving r_n:
;;;         r_n = \prod_{i=1}^{n-1} by_i \cap to_n
;;; where \prod denotes list concatenation. Next calculate the
;;; intersection of by_n with each to_i 0<i<n in turn, concatenating
;;; the result to the corresponding r_i:
;;;         r_i \larrow r_i * (by_n \cap to_i) ; 0<i<n
;;; At this point, the concatenation of all r_n, \prod r_n, is the
;;; result of the intersection between the segments of the by-list and
Joel Rosdahl's avatar
Joel Rosdahl committed
696
;;; to-list we have retrieved so far.
697
698
699
700
701
702
703
704
705
706
707
;;;
;;; If the total size of the result equals or exceeds the size we
;;; requested, finish the loop and return the results.
;;;
;;;
;;; Althogh the solution may seem a bit complicated, it's really not
;;; that bad once you think about how it's done. The good thing about
;;; it is that it does as little work as possible comparing elements.
;;; The bad part is that it constructs a little too many new cons
;;; cells, although reversing most of the lists does help.
;;;
708
;;; The following optimisations have been implemented:
709
;;;
710
711
712
713
714
715
716
717
718
719
720
721
722
;;; - Exploit the fact that the user's map is sorted by aborting the
;;; search if we have the full conference map and the lowest number we
;;; have from the user's map is lower than the lowest number in the
;;; conference map. Highest numbers apply when we are searching from
;;; the front.

;;; +++ FIXME: For users that have written very little in high-traffic
;;; conferences, and where the request cannot be fulfilled (if num is
;;; too high) this function ends up scanning the entire conference
;;; map. At some point it is probably faster to get all the text-stats
;;; in the user's map (when we've retrieved the map we know how many
;;; there are) and look at the recipients rather than calculate the
;;; intersection. 
723
;;;
724

David Byers's avatar
David Byers committed
725
726
(defun lyskom-get-texts-by-and-to (persno confno num 
                                          &optional again pstart cstart)
727
  "Get NUM texts written by person PERSNO with conference CONFNO as a
David Byers's avatar
David Byers committed
728
729
recipient. If optional AGAIN is non-nil, continue from where we were.
Args: persno confno num &optional again pstart cstart"
730
731
  (blocking-do-multiple ((persstat (get-pers-stat persno))
                         (confstat (get-conf-stat confno)))
732
733
734
735
736
737
738
739
740
    (cond

     ;;
     ;; Special case: reviewing to a letterbox or conference we're not
     ;; a member of and have no access to.
     ;;

     ((and (eq lyskom-pers-no persno)
           confstat
741
742
           (or (conf-type->letterbox (conf-stat->conf-type confstat))
               (and (conf-type->rd_prot (conf-stat->conf-type confstat))
743
744
                    (null (map->text-nos 
                           (blocking-do 'get-map
David Byers's avatar
David Byers committed
745
746
                                        (conf-stat->conf-no confstat)
                                        (conf-stat->first-local-no confstat)
747
748
749
750
751
752
753
754
755
756
757
758
759
760
761
762
763
764
765
766
767
768
769
770
771
772
773
774
775
776
777
778
779
780
781
782
783
784
785
786
787
788
789
790
791
792
793
794
795
796
797
798
799
800
801
802
803
804
805
806
807
808
809
810
811
812
813
814
815
816
817
818
819
820
821
822
823
824
825
826
827
828
829
830
831
832
833
834
835
836
837
838
839
840
841
842
843
844
845
846
847
848
                                        1)))))
           (not (eq persno confno)))
      (lyskom-get-letters-to persno confno num again pstart))

     ;;
     ;; General case
     ;;

     (t
      (lyskom-check-review-access confstat persstat)
      (cond ((and again (null num)) (setq num lyskom-last-review-num))
            ((and again (< lyskom-last-review-num 0) (setq num (- num)))))
      (let* ((result-list (if again
                              lyskom-last-review-saved-result-list
                            nil))
             (by-list (if again
                          lyskom-last-review-saved-by-list
                        nil))
             (to-list (if again 
                          lyskom-last-review-saved-to-list 
                        nil))
             (result-size (if again
                              lyskom-last-review-saved-result-size
                            0))
             (by nil)
             (to nil)
             (increment lyskom-fetch-map-nos)
             (plow (or pstart (pers-stat->first-created-text persstat)))
             (phigh (1- (+ plow (pers-stat->no-of-created-texts persstat))))
             (pmark (cond (again lyskom-last-review-pmark)
                          ((and num (< num 0)) plow)
                          (t phigh)))
             (clow (or cstart (conf-stat->first-local-no confstat)))
             (chigh (1- (+ clow (conf-stat->no-of-texts confstat))))
             (cmark (cond (again lyskom-last-review-cmark)
                          ((and num (< num 0)) clow)
                          (t chigh)))
             (smallest (if again lyskom-last-review-saved-smallest nil))
             (largest (if again lyskom-last-review-saved-largest nil))
             (abort-loop nil))

        (if (null num)
            (setq num (1+ phigh)))

        (while (and (or (and (<= pmark phigh)
                             (>= pmark plow))
                        (and (<= cmark chigh)
                             (>= cmark clow)))
                    (> (abs num) result-size)
                    (not abort-loop))

          (setq by (and (<= pmark phigh)
                        (>= pmark plow)
                        (lyskom-remove-zeroes
                         (listify-vector
                          (map->text-nos
                           (blocking-do 'get-created-texts
                                        (pers-stat->pers-no persstat)
                                        (if (< num 0)
                                            pmark
                                          (max 0 (- pmark (1- increment))))
                                        increment)))))
                to (and (<= cmark chigh)
                        (>= cmark clow)
                        (lyskom-remove-zeroes
                         (listify-vector
                          (map->text-nos
                           (blocking-do 'get-map
                                        (conf-stat->conf-no confstat)
                                        (if (< num 0)
                                            cmark
                                          (max 0 (- cmark (1- increment))))
                                        increment))))))

          (if (> num 0)
              (if (and smallest by
                       (> smallest (car by)))
                  (setq abort-loop t))
            (if (and largest by
                     (< largest (car (nthcdr (1- (length by)) by))))
                (setq abort-loop t)))

          ;;
          ;;    Add intersection between new TO and old BYs
          ;;    to the results list.
          ;;

          (setq result-list
                (cons (apply 'nconc
                             (mapcar 
                              (function
                               (lambda (x)
                                 (lyskom-intersection to x)))
                              by-list))
                      result-list))

          ;;
          ;;    Add new BY and TO to the by-list and to-list
          ;;

          (setq by-list (cons by by-list)
                to-list (cons to to-list))
849
      
850

851
852
853
854
855
          ;;
          ;;    Add intersections between new BY and all TOs
          ;;

          (setq result-list
856
857
858
859
860
861
                (lyskom-mapcar2 (function
                                 (lambda (x y)
                                   (lyskom-intersection y
                                                        (nconc x by))))
                                result-list
                                to-list))
862
863
864
865
866
867
868
869
870
871
872
873
874
875
876
877
878
879
880
881
882
883
884
885
886
887
888
889
890
891
892
893
894
895
896
897
898
899
900
901
902
903
904
905
906
907
908
909
910
911
912
913

          (setq result-size (apply '+ (mapcar 'length result-list)))

          ;;
          ;;    Adjust the marks
          ;;

          (if (> num 0)
              (setq pmark (- pmark increment)
                    cmark (- cmark increment))
            (setq pmark (+ pmark increment)
                  cmark (+ cmark increment)))


          ;;
          ;;  If we have exhausted the conference, calculate smallest and
          ;;  largest
          ;;

          (if (and (null smallest)
                   (null largest)
                   (or (> cmark chigh)
                       (< cmark clow)))
              (setq smallest 
                    (apply 'min
                           (mapcar (function (lambda (x)
                                               (if x
                                                   (apply 'min x)
                                                 (lyskom-maxint))))
                                   to-list))
                    largest
                    (apply 'max
                           (mapcar (function (lambda (x)
                                               (if x
                                                   (apply 'max x)
                                                 -1)))
                                   to-list))))


          ;;
          ;;    This is the end of the while loop
          ;;

          )

        (setq lyskom-last-review-pmark pmark)
        (setq lyskom-last-review-cmark cmark)
        (setq lyskom-last-review-saved-by-list by-list)
        (setq lyskom-last-review-saved-to-list to-list)
        (setq lyskom-last-review-saved-smallest smallest)
        (setq lyskom-last-review-saved-largest largest)

914
        ;;
915
        ;;  Extract results
916
        ;;
917

918
        (setq result-list
919
920
921
              (apply 'nconc (if (< num 0)
                                (nreverse result-list)
                              result-list)))
922

923
        ;;
924
        ;;  Save discarded results and return retained results
925
        ;;
926

927
        (if (> num 0)
928
929
930
931
932
933
934
935
            (progn
              (setq lyskom-last-review-saved-result-list
                    (nfirst (- (length result-list) num) result-list))
              (setq lyskom-last-review-saved-result-size
                    (length  lyskom-last-review-saved-result-list))
              (setq lyskom-last-review-saved-result-list
                    (cons lyskom-last-review-saved-result-list
                          (make-list (- (length by-list) 1) nil)))
936
              (nthcdr (max 0 (- (length result-list) num)) result-list))
937

David Byers's avatar
David Byers committed
938
939
940
941
942
943
944
945
          (progn
            (setq lyskom-last-review-saved-result-list 
                  (nthcdr (- num) result-list))
            (setq lyskom-last-review-saved-result-size
                  (length lyskom-last-review-saved-result-list))
            (setq lyskom-last-review-saved-result-list
                  (cons lyskom-last-review-saved-result-list
                        (make-list (- (length by-list) 1) nil)))
946
            (nfirst (- num) result-list))))))))
David Byers's avatar
David Byers committed
947

948

949
950

;;; ===============================================================
951
;;; lyskom-get-texts-by, lyskom-get-texts-to
952
953
;;; Author: David Byers
;;;
954
955
956
957
958
959
960
;;; These functions get data in chunks, starting with the number of
;;; texts requested. If they come up empty (which is common when
;;; scanning from the beginning of a map), the increment is
;;; exponentially increased up to a maximum of 150. 
;;;
;;; lyskom-get-texts-by is also careful to filter out those texts that
;;; are not readable, hence the added complexity in that function.
961
962
;;;

David Byers's avatar
David Byers committed
963
(defun lyskom-get-texts-by (persno num &optional again pstart)
964
  "Get NUM texts written by PERSNO. Args: persno num"
David Byers's avatar
David Byers committed
965
  (let* ((persstat (blocking-do 'get-pers-stat persno)))
966
    (lyskom-check-review-access t persstat)
David Byers's avatar
David Byers committed
967
968
969
970
    (lyskom-get-texts-by-generic persno num nil nil again pstart)))

(defun lyskom-get-texts-by-generic (persno num pred args 
                                           &optional again pstart)
David Byers's avatar
David Byers committed
971
972
973
  "Get NUM texts written by PERSNO. Args: persno num

Cannot be called from a callback."
David Byers's avatar
David Byers committed
974
975
  (let* ((persstat (blocking-do 'get-pers-stat persno))
         (user-area (pers-stat->user-area persstat)))
976

David Byers's avatar
David Byers committed
977
978
979
980
    (cond ((and again (null num)) (setq num lyskom-last-review-num))
          ((and again (< lyskom-last-review-num 0)) (setq num (- num))))

    (let* ((plow (or pstart (pers-stat->first-created-text persstat)))
981
           (phigh (1- (+ plow (pers-stat->no-of-created-texts persstat))))
David Byers's avatar
David Byers committed
982
983
984
           (result (if again
                       lyskom-last-review-saved-result-list
                     nil))
985
           (increment (if num (abs num)))
David Byers's avatar
David Byers committed
986
987
988
           (mark (cond (again lyskom-last-review-pmark)
                        ((and num (< num 0)) plow)
                        (t phigh)))
989
           (collector nil)
990
           (found nil)
991
           (start nil)
992
993
994
995
996
997
998
999
1000
1001
1002
           (data nil))
    

      (if (null num)
          (setq num (1+ phigh)
                mark phigh
                increment (1+ phigh)))

      (while (and (<= mark phigh)
                  (>= mark plow)
                  (> (abs num) (length result)))
1003
1004
1005
1006
1007
1008
1009
1010
1011

        (setq increment (min lyskom-fetch-map-nos increment))
        (setq start (if (< num 0)
                        mark
                      (- mark (1- increment))))
        (if (< start 0)
            (progn
              (setq increment (- increment start))
              (setq start 0)))
1012
      
David Byers's avatar
David Byers committed
1013
1014
1015
1016
1017
1018
1019
1020
        (setq data (delq user-area
                         (lyskom-remove-zeroes
                          (listify-vector
                           (map->text-nos
                            (blocking-do 'get-created-texts
                                         persno
                                         start
                                         increment))))))
1021

1022
1023
1024
1025
1026
1027
        (setq collector (make-collector))
        (mapcar
         (function
          (lambda (x)
            (initiate-get-text-stat 'main 
                                    (function
David Byers's avatar
David Byers committed
1028
1029
1030
1031
1032
1033
                                     (lambda (x collector pred args)
                                       (when (and x
                                                  (or (null pred)
                                                      (apply pred
                                                             x
                                                             args)))
1034
1035
1036
                                           (collector-push 
                                            (text-stat->text-no x)
                                            collector))))
David Byers's avatar
David Byers committed
1037
                                    x collector pred args)))
1038
1039
         data)

1040
        (lyskom-wait-queue 'main)
1041
        (setq found (nreverse (collector->value collector)))
1042

1043
1044
        (if (> num 0)
            (setq result (nconc found result)
1045
                  mark (- mark increment))
1046
1047
1048
          (setq result (nconc result found)
                mark (+ mark increment)))
        (if (null found)
1049
1050
            (setq increment (min lyskom-fetch-map-nos (* increment 2)))
          (setq increment (- (abs num) (length result)))))
1051

David Byers's avatar
David Byers committed
1052
1053
      (setq lyskom-last-review-pmark mark)

1054
      (if (> num 0)
David Byers's avatar
David Byers committed
1055
1056
1057
          (progn
            (setq lyskom-last-review-saved-result-list 
                  (nfirst (- (length result) num) result))
1058
            (nthcdr (max 0 (- (length result) num)) result))
David Byers's avatar
David Byers committed
1059
1060
1061
1062
        (progn
          (setq lyskom-last-review-saved-result-list
                (nthcdr (- num) result))
          (nfirst (- num)  result))))))
1063

David Byers's avatar
David Byers committed
1064
(defun lyskom-get-texts-to (confno num &optional again cstart)
1065
  "From CONFNO get NUM texts."
1066
1067
1068
1069
  (let ((confstat (blocking-do 'get-conf-stat confno)))

    (lyskom-check-review-access confstat t)

David Byers's avatar
David Byers committed
1070
1071
1072
1073
    (cond ((and again (null num)) (setq num lyskom-last-review-num))
          ((and again (< lyskom-last-review-num 0)) (setq num (- num))))

    (let* ((clow (or cstart (conf-stat->first-local-no confstat)))
1074
           (chigh (1- (+ clow (conf-stat->no-of-texts confstat))))
David Byers's avatar
David Byers committed
1075
1076
1077
           (result (if again 
                       lyskom-last-review-saved-result-list
                     nil))
1078
           (start nil)
1079
           (increment (and num (abs num)))
David Byers's avatar
David Byers committed
1080
1081
1082
           (mark (cond (again lyskom-last-review-cmark)
                       ((and num (< num 0)) clow)
                       (t chigh))))
1083
1084
1085
1086
1087
1088
1089
1090
1091

      (if (null num)
          (setq num (1+ chigh)
                increment (1+ chigh)
                mark chigh))

      (while (and (<= mark chigh)
                  (>= mark clow)
                  (> (abs num) (length result)))
1092
1093
1094
1095
1096
1097
1098
1099
1100
1101

        (setq increment (min lyskom-fetch-map-nos increment))
        (setq start (if (< num 0)
                        mark
                      (- mark (1- increment))))
        (if (< start 0)
            (progn
              (setq increment (- increment start))
              (setq start 0)))

1102
1103
1104
1105
1106
        (let ((found (lyskom-remove-zeroes
                      (listify-vector
                       (map->text-nos
                        (blocking-do 'get-map
                                     confno
1107
                                     start
1108
1109
1110
1111
1112
1113
1114
1115
1116
1117
                                     increment))))))
          (if (> num 0)
              (setq result (nconc found result)
                    mark (- mark increment)
                    increment (- (abs num) (length result)))
            (setq result (nconc result found)
                  mark (+ mark increment)))
          (if (null found)
              (setq increment (min lyskom-fetch-map-nos (* increment 2)))
            (setq increment (- (abs num) (length result))))))
1118

David Byers's avatar
David Byers committed
1119
1120
      (setq lyskom-last-review-cmark mark)

1121
      (if (> num 0)
David Byers's avatar
David Byers committed
1122
1123
1124
          (progn
            (setq lyskom-last-review-saved-result-list 
                  (nfirst (- (length result) num) result))
1125
            (nthcdr (max 0 (- (length result) num)) result))
David Byers's avatar
David Byers committed
1126
1127
1128
1129
        (progn
          (setq lyskom-last-review-saved-result-list
                (nthcdr (- num) result))
          (nfirst (- num)  result))))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1130
1131


David Byers's avatar
David Byers committed
1132
1133


David Byers's avatar
David Byers committed
1134
1135
1136
1137
1138
;;; ============================================================
;;;                 terse baklnges
;;;

(def-kom-command kom-review-backward ()
1139
1140
1141
1142
1143
  "Toggles the reviewing order. If you are currently reviewing texts
oldest to newest, review newest to oldest instead. When reviewing
newest to oldest, change to oldest to newest.

See `kom-review-uses-cache', `kom-review-priority' and
1144
`kom-review-marks-texts-as-read' for information on settings that affect
1145
all review-related functions."
Per Cederqvist's avatar
.    
Per Cederqvist committed
1146
1147
1148
  (interactive)
  (cond
   ((and (not (read-list-isempty lyskom-reading-list))
1149
	 (memq (read-info->type (read-list->first lyskom-reading-list))
1150
		 '(REVIEW REVIEW-MARK)))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1151
1152
    (let* ((info (read-list->first lyskom-reading-list))
	   (list (read-info->text-list info))
1153
	   (texts (text-list->texts list))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1154
	   (forward (read-info->forward info)))
David Byers's avatar
David Byers committed
1155
      (set-text-list->texts list (nreverse texts))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1156
      (set-read-info->forward info (not forward))
1157
1158
1159
1160
      (lyskom-format-insert 'you-review 
			    (lyskom-get-string (if (not forward)
						   'forward
						 'backward)))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1161
   (t
David Byers's avatar
David Byers committed
1162
    (lyskom-insert-string 'illegal-command))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1163
1164
1165
1166
1167
1168
1169
1170


;;; ================================================================
;;;                   ]terse tr{det - review tree

;;; Author: Linus Tolke


David Kågedal's avatar
David Kågedal committed
1171
(def-kom-command kom-review-tree (&optional text-no)
1172
1173
1174
1175
1176
1177
1178
1179
1180
  "Recursively review all comments to the selected text.
This command will descend recursively in the comment tree, as when
reading texts normally. Unlike when reading normally, filters are
not applied and circular structures are not dealt with gracefully.

This command accepts text number prefix arguments \(see
`lyskom-read-text-no-prefix-arg').

See `kom-review-uses-cache', `kom-review-priority' and
1181
`kom-review-marks-texts-as-read' for information on settings that affect
1182
all review-related functions."
1183
  (interactive (list (lyskom-read-text-no-prefix-arg 'review-tree-q)))
David Kågedal's avatar
David Kågedal committed
1184
1185
  (lyskom-tell-internat 'kom-tell-review)
  (if text-no
1186
1187
1188
1189
1190
1191
1192
1193
1194
      (progn
        (unless kom-review-uses-cache
          (cache-del-text-stat text-no))
        
        (let ((ts (blocking-do 'get-text-stat text-no)))
          (lyskom-follow-comments ts
                                  nil 'review
                                  (lyskom-review-get-priority)
                                  t)))
David Kågedal's avatar
David Kågedal committed
1195
1196
1197
    (lyskom-insert-string 'read-text-first)))


David Byers's avatar
David Byers committed
1198
1199
1200
1201
1202
1203
1204
1205
1206
1207
1208
1209
1210
1211
1212
1213
1214
1215
1216
1217
1218
1219
1220
1221
1222
1223
1224
1225
1226
1227
1228
1229
1230
1231
1232
1233
(def-kom-command kom-unread-tree (text-no)
    "Recursively mark all comments to the selected text as unread.

This command accepts text number prefix arguments \(see
`lyskom-read-text-no-prefix-arg')."
  (interactive (list (lyskom-read-text-no-prefix-arg 'unread-tree-q)))
  (lyskom-unread-tree text-no nil))

(defun lyskom-unread-tree (text-no visited-list)
  "Perform the function of kom-unread-tree."
  (let* ((text-stat (blocking-do 'get-text-stat text-no))
         (worklist (and text-stat
                        (lyskom-text-comments text-stat))))
    (if (null (car worklist))
        (lyskom-format-insert 'no-such-text-no text-no)
      (while worklist
        (let ((cur (car worklist)))
          (setq worklist (cdr worklist))
          (unless (memq cur visited-list)
            (setq visited-list (cons cur visited-list)
                  cur (blocking-do 'get-text-stat cur))
            (when cur
              (when (delq nil (mapcar 'lyskom-get-membership
                                      (lyskom-text-recipients cur)))
                (lyskom-format-insert 'marking-text-unread
                                      (text-stat->text-no cur))
                (lyskom-report-command-answer
                 (lyskom-mark-unread (text-stat->text-no cur))))
              (lyskom-traverse text-no (lyskom-text-comments cur)
                (unless (or (memq text-no visited-list)
                            (memq text-no worklist)
                  (setq worklist (cons text-no worklist)))))))))))
  visited-list)



1234
(def-kom-command kom-find-root (text-no)
1235
1236
1237
1238
1239
1240
1241
1242
  "Finds the root text of the tree containing the selected text.
When there is more than one root, all will be included in a review
operation.

This command accepts text number prefix arguments \(see
`lyskom-read-text-no-prefix-arg').

See `kom-review-uses-cache', `kom-review-priority' and
1243
`kom-review-marks-texts-as-read' for information on settings that affect
1244
all review-related functions."
1245
  (interactive (list (lyskom-read-text-no-prefix-arg 'find-root-q)))
David Kågedal's avatar
David Kågedal committed
1246
1247
  (lyskom-tell-internat 'kom-tell-review)
  (cond
1248
   (text-no
1249
1250
    (unless kom-review-uses-cache
      (cache-del-text-stat text-no))
1251
    (let* ((ts (blocking-do 'get-text-stat text-no))
David Byers's avatar
David Byers committed
1252
1253
1254
	   (r (lyskom-find-root ts t)))
      (cond ((> (length r) 1)
             (lyskom-format-insert-before-prompt
1255
1256
1257
1258
1259
1260
1261
1262
              (lyskom-get-string 'more-than-one-root) ts)
             (lyskom-review-enter-read-info
              (lyskom-create-read-info
               'REVIEW
               nil
               (lyskom-review-get-priority)
               (lyskom-create-text-list r)
               nil t) t))
1263
1264
1265
            (r (unless kom-review-uses-cache
                 (cache-del-text-stat (car r)))
               (lyskom-view-text (car r)))
1266
            (t (lyskom-insert-string 'could-not-find-root)))
David Kågedal's avatar
David Kågedal committed
1267
1268
1269
      )
    )
   (t
1270
    (lyskom-insert-string 'confusion-what-to-find-root))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1271

David Byers's avatar
David Byers committed
1272
1273
1274
1275
1276
1277
1278
1279
1280
1281
(def-kom-command kom-unread-root (text-no)
  "Finds the root text of the tree containing the selected text
and marks it as unread.

This command accepts text number prefix arguments \(see
`lyskom-read-text-no-prefix-arg')."
  (interactive (list (lyskom-read-text-no-prefix-arg 'unread-root-q)))
  (if text-no
      (let* ((ts (blocking-do 'get-text-stat text-no))
             (r (lyskom-find-root ts t)))
1282
1283
1284
1285
1286
1287
        (if r
            (lyskom-traverse text-no r
              (lyskom-format-insert 'marking-text-unread text-no)
              (lyskom-report-command-answer
               (lyskom-mark-unread text-no)))
          (lyskom-insert-string 'could-not-find-root)))
David Byers's avatar
David Byers committed
1288
1289
    (lyskom-insert-string 'confusion-what-to-unread-root)))

Per Cederqvist's avatar
.    
Per Cederqvist committed
1290

1291
(def-kom-command kom-find-root-review (text-no)
1292
1293
1294
1295
1296
1297
1298
1299
1300
  "Finds the root of the comment tree containing the selected texts
and then recursively reviews all its comments. For texts with a single
root, this is equivalent to doing `kom-find-root' followed by
`kom-review-tree'.

This command accepts text number prefix arguments \(see
`lyskom-read-text-no-prefix-arg').

See `kom-review-uses-cache', `kom-review-priority' and
1301
`kom-review-marks-texts-as-read' for information on settings that affect
1302
all review-related functions."
1303
  (interactive (list (lyskom-read-text-no-prefix-arg 'find-root-review-q)))
1304
  (lyskom-tell-internat 'kom-tell-review)
Per Cederqvist's avatar
.    
Per Cederqvist committed
1305
  (cond
1306
   (text-no
1307
1308
    (unless kom-review-uses-cache
      (cache-del-text-stat text-no))
1309
    (let* ((ts (blocking-do 'get-text-stat text-no))
David Byers's avatar
David Byers committed
1310
1311
1312
1313
1314
1315
           (start (lyskom-find-root ts t)))
      (cond ((> (length start) 1)
             (lyskom-format-insert-before-prompt
              (lyskom-get-string 'more-than-one-root-review) ts)
             (lyskom-review-tree (car start)))
            (start (lyskom-review-tree (car start)))
1316
            (t (lyskom-insert-string 'could-not-find-root)))))
1317
   (t (lyskom-insert-string 'confusion-what-to-find-root-review))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1318

David Byers's avatar
David Byers committed
1319
1320
1321
1322
1323
1324
1325
1326
1327
1328
1329
1330
1331
1332
1333
1334
1335
(def-kom-command kom-unread-root-review (text-no)
  "Finds the root of the comment tree containing the selected texts
and then recursively marks all its comments as unread. For texts with
a single root, this is equivalent to doing `kom-find-root' followed by
`kom-unread-tree'.

This command accepts text number prefix arguments \(see
`lyskom-read-text-no-prefix-arg')."
  (interactive (list (lyskom-read-text-no-prefix-arg 'unread-root-review-q)))
  (if text-no
      (let* ((ts (blocking-do 'get-text-stat text-no))
             (start (lyskom-find-root ts t))
             (marked nil))
        (lyskom-traverse text-no start
          (setq marked (lyskom-unread-tree text-no marked))))
    (lyskom-insert-string 'confusion-what-to-unread-root-review)))

Per Cederqvist's avatar
.    
Per Cederqvist committed
1336

David Byers's avatar
X    
David Byers committed
1337
(defun lyskom-find-root (text-stat &optional all)
Per Cederqvist's avatar
.    
Per Cederqvist committed
1338
  "Finds the root text of the tree containing the text TEXT-STAT.
David Byers's avatar
X    
David Byers committed
1339
1340
1341
1342
1343
1344
1345
Args: TEXT-STAT &optional ALL
If ALL is set, return a list of all root texts."  
  (cond (text-stat
         (let ((queue (list text-stat))
               (head nil)
               (misclist nil)
               (tmp nil)
1346
               (seen nil)
David Byers's avatar
X    
David Byers committed
1347
1348
1349
1350
1351
               (result nil))
           (while queue
             (setq head (car queue))
             (setq queue (cdr queue))
             (setq tmp nil)
1352

David Byers's avatar
X    
David Byers committed
1353
1354
1355
             ;;
             ;; For each parent, add it to the queue
             ;;
1356

David Byers's avatar
X    
David Byers committed
1357
1358
1359
             (setq misclist (text-stat->misc-info-list head))
             (while misclist
               (cond ((eq (misc-info->type (car misclist)) 'COMM-TO)
1360
1361
1362
1363
1364
1365
1366
1367
1368
                      (if (memq (misc-info->comm-to (car misclist)) seen)
                          (setq tmp (cons t tmp))
                        (unless kom-review-uses-cache
                          (cache-del-text-stat (misc-info->comm-to (car misclist))))
                        (setq seen (cons (misc-info->comm-to (car misclist)) seen)
                              tmp (cons
                                   (blocking-do 'get-text-stat
                                                (misc-info->comm-to (car misclist)))
                                   tmp))))
David Byers's avatar
X    
David Byers committed
1369
                     ((eq (misc-info->type (car misclist)) 'FOOTN-TO)
1370
1371
1372
1373
1374
1375
1376
1377
1378
                      (if (memq (misc-info->footn-to (car misclist)) seen)
                          (setq tmp (cons t tmp))
                        (unless kom-review-uses-cache
                          (cache-del-text-stat (misc-info->footn-to (car misclist))))
                        (setq seen (cons (misc-info->footn-to (car misclist)) seen)
                              tmp (cons
                                   (blocking-do 'get-text-stat
                                                (misc-info->footn-to (car misclist)))
                                   tmp)))))
David Byers's avatar
X    
David Byers committed
1379
               (setq misclist (cdr misclist)))
1380

1381
1382
1383
             ;;
             ;; Remove unreadable texts
             ;;
1384

1385
             (setq tmp (delq nil tmp))
1386

David Byers's avatar
X    
David Byers committed
1387
1388
1389
1390
1391
1392
             ;;
             ;; If no parents were found, this is is a top-level text
             ;;

             (when (null tmp)
               (setq result (cons head result))
1393
1394
1395
1396
1397
1398
1399
               (unless all (setq queue nil)))

             ;;
             ;; Remove "I visited this parent already" markers
             ;;

             (setq tmp (delq t tmp))
David Byers's avatar
X    
David Byers committed
1400
             (setq queue (nconc tmp queue)))
1401
1402
           (if all 
               (mapcar 'text-stat->text-no result)
David Byers's avatar
X    
David Byers committed
1403
1404
1405
1406
             (text-stat->text-no (car result)))))
        (t nil)))


Per Cederqvist's avatar
.    
Per Cederqvist committed
1407
1408
1409
1410
1411


(defun lyskom-review-tree (text)
  "Takes a TEXT as an arg, shows the text and the tree of all comments.
Does a lyskom-end-of-command.
Linus Tolke's avatar
Linus Tolke committed
1412
Text is a text-no."
Per Cederqvist's avatar
.    
Per Cederqvist committed
1413
1414
  (cond
   ((integerp text)
1415
1416
    (unless kom-review-uses-cache
      (cache-del-text-stat text))
1417
    (lyskom-view-text text nil t nil (lyskom-review-get-priority) t))
Linus Tolke's avatar
Linus Tolke committed
1418
1419
1420
1421
   (t
    (signal 'lyskom-internal-error
	    (list 'lyskom-review-tree
		  "Called with incorrect argument."
David Kågedal's avatar
David Kågedal committed
1422
		  text)))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1423
1424
1425
1426
1427
1428
1429
1430


;;; ================================================================
;;;                     ]terse n{sta -  review next

;;; Author: Linus Tolke


David Byers's avatar
David Byers committed
1431
(def-kom-command kom-review-next ()
1432
1433
1434
1435
1436
1437
  "Resumes an interupted review by moving all suspended review entries
to the front. This command may be less-than-useful depending on how
review priorities are set, and the setting of
`kom-higher-priority-breaks'.

See `kom-review-uses-cache', `kom-review-priority' and