review.el 49.2 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, 1996  Lysator Academic Computer Association.
Linus Tolke's avatar
Linus Tolke committed
5
6
7
8
9
;;;;;
;;;;; This file is part of the LysKOM server.
;;;;; 
;;;;; 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"))

46
47
48
49
50
(put 'lyskom-cant-review-error
     'error-conditions
     '(error lyskom-error lyskom-review-error))
(put 'lyskom-review-error 'error-message
     "Review error")
51

52

David Byers's avatar
David Byers committed
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
;;; ======================================================================
;;; State-saving variables
;;;

(defvar lyskom-last-review-by nil)
(defvar lyskom-last-review-to nil)
(defvar lyskom-last-review-num nil)
(defvar lyskom-last-review-pmark nil)
(defvar lyskom-last-review-cmark nil)
(defvar lyskom-last-review-saved-result-list nil)
(defvar lyskom-last-review-saved-by-list nil)
(defvar lyskom-last-review-saved-to-list nil)
(defvar lyskom-last-review-saved-result-size 0)
(defvar lyskom-last-review-saved-smallest nil)
(defvar lyskom-last-review-saved-largest nil)
(defvar lyskom-have-review nil)


71
72
73
74
75
(defun lyskom-remove-zeroes (a)
  "Returns a copy of list where all zeroes are removed."
  (delq 0 (copy-sequence a)))


Per Cederqvist's avatar
.    
Per Cederqvist committed
76
77
78
;;; ================================================================
;;;              ]terse av, till - Review by X to Conference Y.

David Byers's avatar
David Byers committed
79
;;; Author: David Byers
80

Per Cederqvist's avatar
.    
Per Cederqvist committed
81

82
83
84
85
86
87
(def-kom-command kom-review-all ()
  "Review every articles of an author written to a conference."
  (interactive)
  (lyskom-tell-internat 'kom-tell-review)
  (lyskom-review-by-to 0))

Per Cederqvist's avatar
.    
Per Cederqvist committed
88

David Byers's avatar
David Byers committed
89
(def-kom-command kom-review-more (&optional count)
90
  "Review more articles using the same critera as the last review."
91
  (interactive "p")
David Byers's avatar
David Byers committed
92
93
  (if (not lyskom-have-review)
      (lyskom-format-insert 'no-review-done)
94
95
96
97
    (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
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
           (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
127
128
129
130
131
132
133
                (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
134
135
136
137
138
139
140
              (lyskom-insert-string 'no-such-text)))
        (lyskom-review-error (if arg
                                 nil
                               (lyskom-insert-string 'no-such-text)))))))

                       

141
142
143
144
145
146
147
148
149
150
(def-kom-command kom-review-first (&optional count)
  "Reviews all articles of author that is written to conference recipient.
If return is given instead of an author then all authors to that conference is
shown. If return is given instead of conference then all conferences for that
person is chosen.
If a negative numeric argument is given then only the last COUNT articles are 
chosen. If the argument is positive then the first -COUNT articles are chosen.
If the argument is zero the all articles are chosen.
No argument is equivalent to COUNT 1.
The defaults for this command is the conference that you are in."
151
  (interactive "P")
152
153
154
155
156
  (lyskom-tell-internat 'kom-tell-review)
  (lyskom-review-by-to (- (or count
                           (lyskom-read-number
                            (lyskom-get-string 'review-how-many) 1)))))

157

David Kågedal's avatar
David Kågedal committed
158
(def-kom-command kom-review-by-to (&optional count)
Per Cederqvist's avatar
.    
Per Cederqvist committed
159
160
161
162
163
164
165
166
167
  "Reviews all articles of author that is written to conference recipient.
If return is given instead of an author then all authors to that conference is
shown. If return is given instead of conference then all conferences for that
person is chosen.
If a positive numeric argument is given then only the last COUNT articles are 
chosen. If the argument is negative then the first -COUNT articles are chosen.
If the argument is zero the all articles are chosen.
No argument is equivalent to COUNT 1.
The defaults for this command is the conference that you are in."
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
  (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)
183
                             (lyskom-get-string 'everything))
184
185
186
187
188
189
190
                            ((> count 0)
                             (lyskom-format 'latest-n count))
                            ((< count 0)
                             (lyskom-format 'first-n
                                            (- count))))))
         (by (lyskom-read-conf-no 
              (lyskom-format 'review-info (lyskom-format 'info-by-whom info))
191
              '(pers) t nil t))
192
193
194
         (to (lyskom-read-conf-no 
              (lyskom-format 'review-info
                             (lyskom-format 'info-to-conf info))
195
              '(all) 
196
              t
197
              ;; If person is not given we must give
198
199
              ;; conf  -- Not anymore!
              ;; (not (zerop by))
David Byers's avatar
David Byers committed
200
201
              (if (or (null lyskom-current-conf)
                      (zerop lyskom-current-conf))
202
                  ""
203
204
205
                (cons (conf-stat->name
                         (blocking-do 'get-conf-stat
                                      lyskom-current-conf)) 0))
206
              t)))
David Kågedal's avatar
David Kågedal committed
207
208

    (if (not (zerop to))
209
        (cache-del-conf-stat to))
David Kågedal's avatar
David Kågedal committed
210
    (if (not (zerop by)) 
211
        (cache-del-pers-stat by))
212

David Byers's avatar
David Byers committed
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
    (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
234

235
236
237
    (condition-case arg
        (let ((list (lyskom-get-texts-by-to by to count)))
          (if list
238
239
240
241
242
243
244
              (lyskom-review-enter-read-info
               (lyskom-create-read-info
                'REVIEW
                nil
                (lyskom-review-get-priority)
                (lyskom-create-text-list list)
                nil t) t)
245
246
247
248
            (lyskom-insert-string 'no-such-text)))
      (lyskom-review-error (if arg
                               nil 
                             (lyskom-insert-string 'no-such-text))))))
249

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

251
252
253
254
255
256
257
258
259
260
;;; ================================================================
;;; 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.
;;;


David Byers's avatar
David Byers committed
261
262
(defun lyskom-get-texts-by-to (by to num &optional again)
  "Get NUM texts written by person number BY in conference number TO
263
Args: BY TO NUM"
264
  (cond ((and (zerop by) 
David Byers's avatar
David Byers committed
265
266
267
              (zerop to)) (lyskom-get-texts-globally num again))
        ((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
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
        ((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
285
286
                           (and (memq (misc-info->type misc)
					lyskom-recpt-types-list)
David Byers's avatar
David Byers committed
287
288
289
290
291
                                (eq (misc-info->recipient-no misc) to)))))
               found)))
          (list to)
          again))
         (t (lyskom-get-texts-by-and-to by to num again))))
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327


;;; ============================================================
;;; 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))))))

328
329


330
331
332
333
334
;;; ============================================================
;;; lyskom-get-texts-globally
;;; Author: Per Cederquist, David Byers
;;;

David Byers's avatar
David Byers committed
335
336
337
338
339
340
341
342
343
344
345
(defun lyskom-get-texts-globally (num &optional again)
  "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) 
         (lyskom-format-insert 'cant-review-everything)
         (signal 'lyskom-cant-review-error t)))

346
  (let ((result nil)
David Byers's avatar
David Byers committed
347
348
349
        (textno (cond (again lyskom-last-review-cmark)
                      ((< num 0) 1)
                      (t (lyskom-maxint))))
350
351
352
353
354
        (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
355
                (not (null textno))
356
357
                (< (length result) (abs num)))
      (setq textno (blocking-do op textno))
David Byers's avatar
David Byers committed
358
359
360
      (if textno
          (setq result (cons textno result))))
    (setq lyskom-last-review-cmark textno)
361
362
363
364
    (if (< num 0)
        (nreverse result)
      result)))

365

366
367
368
369
370
371
372
373
374
375
376
377
;;; ================================================================
;;; 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
378
379
380
  "Get NUM texts written by PERSNO. Args: persno num

Cannot be called from a callback."
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
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
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
  (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))
462
            (nthcdr (max 0 (- (length result) num)) result))
463
464
465
466
467
468
469
470
471
472
        (progn
          (setq lyskom-last-review-saved-result-list
                (nthcdr (- num) result))
          (nfirst (- num)  result))))))






473
474
;;; ================================================================
;;; lyskom-get-texts-by-and-to
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
;;; 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
503
;;; to-list we have retrieved so far.
504
505
506
507
508
509
510
511
512
513
514
;;;
;;; 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.
;;;
515
;;; The following optimisations have been implemented:
516
;;;
517
518
519
520
521
522
523
524
525
526
527
528
529
;;; - 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. 
530
;;;
531

David Byers's avatar
David Byers committed
532
533
(defun lyskom-get-texts-by-and-to (persno confno num 
                                          &optional again pstart cstart)
534
  "Get NUM texts written by person PERSNO with conference CONFNO as a
David Byers's avatar
David Byers committed
535
536
recipient. If optional AGAIN is non-nil, continue from where we were.
Args: persno confno num &optional again pstart cstart"
537
538
  (blocking-do-multiple ((persstat (get-pers-stat persno))
                         (confstat (get-conf-stat confno)))
539
540
541
542
543
544
545
546
547
    (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
548
549
           (or (conf-type->letterbox (conf-stat->conf-type confstat))
               (and (conf-type->rd_prot (conf-stat->conf-type confstat))
550
551
                    (null (map->text-nos 
                           (blocking-do 'get-map
David Byers's avatar
David Byers committed
552
553
                                        (conf-stat->conf-no confstat)
                                        (conf-stat->first-local-no confstat)
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
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
655
                                        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))
656
      
657

658
659
660
661
662
663
664
665
666
667
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
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
          ;;
          ;;    Add intersections between new BY and all TOs
          ;;

          (setq result-list
                (mapcar2 (function
                          (lambda (x y)
                            (lyskom-intersection y
                                                 (nconc x by))))
                         result-list
                         to-list))

          (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)

721
        ;;
722
        ;;  Extract results
723
        ;;
724

725
        (setq result-list
726
727
728
              (apply 'nconc (if (< num 0)
                                (nreverse result-list)
                              result-list)))
729

730
        ;;
731
        ;;  Save discarded results and return retained results
732
        ;;
733

734
        (if (> num 0)
735
736
737
738
739
740
741
742
            (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)))
743
              (nthcdr (max 0 (- (length result-list) num)) result-list))
744

David Byers's avatar
David Byers committed
745
746
747
748
749
750
751
752
          (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)))
753
            (nfirst (- num) result-list))))))))
David Byers's avatar
David Byers committed
754

755

756
757

;;; ===============================================================
758
;;; lyskom-get-texts-by, lyskom-get-texts-to
759
760
;;; Author: David Byers
;;;
761
762
763
764
765
766
767
;;; 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.
768
769
;;;

David Byers's avatar
David Byers committed
770
(defun lyskom-get-texts-by (persno num &optional again pstart)
771
  "Get NUM texts written by PERSNO. Args: persno num"
David Byers's avatar
David Byers committed
772
  (let* ((persstat (blocking-do 'get-pers-stat persno)))
773
    (lyskom-check-review-access t persstat)
David Byers's avatar
David Byers committed
774
775
776
777
    (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
778
779
780
  "Get NUM texts written by PERSNO. Args: persno num

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

David Byers's avatar
David Byers committed
784
785
786
787
    (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)))
788
           (phigh (1- (+ plow (pers-stat->no-of-created-texts persstat))))
David Byers's avatar
David Byers committed
789
790
791
           (result (if again
                       lyskom-last-review-saved-result-list
                     nil))
792
           (increment (if num (abs num)))
David Byers's avatar
David Byers committed
793
794
795
           (mark (cond (again lyskom-last-review-pmark)
                        ((and num (< num 0)) plow)
                        (t phigh)))
796
           (collector nil)
797
           (found nil)
798
           (start nil)
799
800
801
802
803
804
805
806
807
808
809
           (data nil))
    

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

      (while (and (<= mark phigh)
                  (>= mark plow)
                  (> (abs num) (length result)))
810
811
812
813
814
815
816
817
818

        (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)))
819
      
David Byers's avatar
David Byers committed
820
821
822
823
824
825
826
827
        (setq data (delq user-area
                         (lyskom-remove-zeroes
                          (listify-vector
                           (map->text-nos
                            (blocking-do 'get-created-texts
                                         persno
                                         start
                                         increment))))))
828

829
830
831
832
833
834
        (setq collector (make-collector))
        (mapcar
         (function
          (lambda (x)
            (initiate-get-text-stat 'main 
                                    (function
David Byers's avatar
David Byers committed
835
836
837
838
839
840
                                     (lambda (x collector pred args)
                                       (when (and x
                                                  (or (null pred)
                                                      (apply pred
                                                             x
                                                             args)))
841
842
843
                                           (collector-push 
                                            (text-stat->text-no x)
                                            collector))))
David Byers's avatar
David Byers committed
844
                                    x collector pred args)))
845
846
         data)

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

850
851
        (if (> num 0)
            (setq result (nconc found result)
852
                  mark (- mark increment))
853
854
855
          (setq result (nconc result found)
                mark (+ mark increment)))
        (if (null found)
856
857
            (setq increment (min lyskom-fetch-map-nos (* increment 2)))
          (setq increment (- (abs num) (length result)))))
858

David Byers's avatar
David Byers committed
859
860
      (setq lyskom-last-review-pmark mark)

861
      (if (> num 0)
David Byers's avatar
David Byers committed
862
863
864
          (progn
            (setq lyskom-last-review-saved-result-list 
                  (nfirst (- (length result) num) result))
865
            (nthcdr (max 0 (- (length result) num)) result))
David Byers's avatar
David Byers committed
866
867
868
869
        (progn
          (setq lyskom-last-review-saved-result-list
                (nthcdr (- num) result))
          (nfirst (- num)  result))))))
870

David Byers's avatar
David Byers committed
871
(defun lyskom-get-texts-to (confno num &optional again cstart)
872
  "From CONFNO get NUM texts."
873
874
875
876
  (let ((confstat (blocking-do 'get-conf-stat confno)))

    (lyskom-check-review-access confstat t)

David Byers's avatar
David Byers committed
877
878
879
880
    (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)))
881
           (chigh (1- (+ clow (conf-stat->no-of-texts confstat))))
David Byers's avatar
David Byers committed
882
883
884
           (result (if again 
                       lyskom-last-review-saved-result-list
                     nil))
885
           (start nil)
886
           (increment (and num (abs num)))
David Byers's avatar
David Byers committed
887
888
889
           (mark (cond (again lyskom-last-review-cmark)
                       ((and num (< num 0)) clow)
                       (t chigh))))
890
891
892
893
894
895
896
897
898

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

      (while (and (<= mark chigh)
                  (>= mark clow)
                  (> (abs num) (length result)))
899
900
901
902
903
904
905
906
907
908

        (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)))

909
910
911
912
913
        (let ((found (lyskom-remove-zeroes
                      (listify-vector
                       (map->text-nos
                        (blocking-do 'get-map
                                     confno
914
                                     start
915
916
917
918
919
920
921
922
923
924
                                     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))))))
925

David Byers's avatar
David Byers committed
926
927
      (setq lyskom-last-review-cmark mark)

928
      (if (> num 0)
David Byers's avatar
David Byers committed
929
930
931
          (progn
            (setq lyskom-last-review-saved-result-list 
                  (nfirst (- (length result) num) result))
932
            (nthcdr (max 0 (- (length result) num)) result))
David Byers's avatar
David Byers committed
933
934
935
936
        (progn
          (setq lyskom-last-review-saved-result-list
                (nthcdr (- num) result))
          (nfirst (- num)  result))))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
937
938


David Byers's avatar
David Byers committed
939
940


David Byers's avatar
David Byers committed
941
942
943
944
945
;;; ============================================================
;;;                 Återse baklänges
;;;

(def-kom-command kom-review-backward ()
Per Cederqvist's avatar
.    
Per Cederqvist committed
946
947
948
949
950
  "Toggles the reviewing order.
If reading forward then starts reading backward and the other way round."
  (interactive)
  (cond
   ((and (not (read-list-isempty lyskom-reading-list))
951
	 (memq (read-info->type (read-list->first lyskom-reading-list))
952
		 '(REVIEW REVIEW-MARK)))
Per Cederqvist's avatar
.    
Per Cederqvist committed
953
954
955
956
    (let* ((info (read-list->first lyskom-reading-list))
	   (list (read-info->text-list info))
	   (texts (cdr list))
	   (forward (read-info->forward info)))
957
      (setcdr list (nreverse texts))
Per Cederqvist's avatar
.    
Per Cederqvist committed
958
      (set-read-info->forward info (not forward))
959
960
961
962
      (lyskom-format-insert 'you-review 
			    (lyskom-get-string (if (not forward)
						   'forward
						 'backward)))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
963
   (t
David Byers's avatar
David Byers committed
964
    (lyskom-insert-string 'illegal-command))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
965
966
967
968
969
970
971
972


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

;;; Author: Linus Tolke


David Kågedal's avatar
David Kågedal committed
973
(def-kom-command kom-review-tree (&optional text-no)
Per Cederqvist's avatar
.    
Per Cederqvist committed
974
975
976
977
978
  "Review all comments to this text.
Descends recursively in the comment-tree without marking the texts as read.
The tree is forgotten when a kom-go-to-next-conf command is issued.
If optional prefix argument TEXT-NO is present view tree from that text 
instead. In this case the text TEXT-NO is first shown." 
979
  (interactive (list (lyskom-read-text-no-prefix-arg 'review-tree-q)))
David Kågedal's avatar
David Kågedal committed
980
981
  (lyskom-tell-internat 'kom-tell-review)
  (if text-no
982
983
984
985
986
987
988
989
990
      (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
991
992
993
    (lyskom-insert-string 'read-text-first)))


994
(def-kom-command kom-find-root (text-no)
Per Cederqvist's avatar
.    
Per Cederqvist committed
995
  "Finds the root text of the tree containing the text in lyskom-current-text."
996
  (interactive (list (lyskom-read-text-no-prefix-arg 'find-root-q)))
David Kågedal's avatar
David Kågedal committed
997
998
  (lyskom-tell-internat 'kom-tell-review)
  (cond
999
   (text-no
1000
1001
    (unless kom-review-uses-cache
      (cache-del-text-stat text-no))
1002
    (let* ((ts (blocking-do 'get-text-stat text-no))
David Byers's avatar
David Byers committed
1003
1004
1005
	   (r (lyskom-find-root ts t)))
      (cond ((> (length r) 1)
             (lyskom-format-insert-before-prompt
1006
1007
1008
1009
1010
1011
1012
1013
              (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))
1014
1015
1016
            (r (unless kom-review-uses-cache
                 (cache-del-text-stat (car r)))
               (lyskom-view-text (car r)))
David Byers's avatar
David Byers committed
1017
            (t (signal 'lyskom-internal-error "Could not find root")))
David Kågedal's avatar
David Kågedal committed
1018
1019
1020
      )
    )
   (t
1021
    (lyskom-insert-string 'confusion-what-to-find-root))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1022
1023


1024
(def-kom-command kom-find-root-review (text-no)
Per Cederqvist's avatar
.    
Per Cederqvist committed
1025
1026
  "Finds the root text of the tree containing the text in lyskom-current-text and
reviews the whole tree in deep-first order."
1027
  (interactive (list (lyskom-read-text-no-prefix-arg 'find-root-review-q)))
1028
  (lyskom-tell-internat 'kom-tell-review)
Per Cederqvist's avatar
.    
Per Cederqvist committed
1029
  (cond
1030
   (text-no
1031
1032
    (unless kom-review-uses-cache
      (cache-del-text-stat text-no))
1033
    (let* ((ts (blocking-do 'get-text-stat text-no))
David Byers's avatar
David Byers committed
1034
1035
1036
1037
1038
1039
1040
           (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)))
            (t (signal 'lyskom-internal-error "Could not find root")))))
1041
   (t (lyskom-insert-string 'confusion-what-to-find-root-review))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1042
1043


David Byers's avatar
X    
David Byers committed
1044
(defun lyskom-find-root (text-stat &optional all)
Per Cederqvist's avatar
.    
Per Cederqvist committed
1045
  "Finds the root text of the tree containing the text TEXT-STAT.
David Byers's avatar
X    
David Byers committed
1046
1047
1048
1049
1050
1051
1052
1053
1054
1055
1056
1057
1058
1059
1060
1061
1062
1063
1064
1065
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)
               (result nil))
           (while queue
             (setq head (car queue))
             (setq queue (cdr queue))
             (setq tmp nil)
             
             ;;
             ;; For each parent, add it to the queue
             ;;
               
             (setq misclist (text-stat->misc-info-list head))
             (while misclist
               (cond ((eq (misc-info->type (car misclist)) 'COMM-TO)
1066
1067
                      (unless kom-review-uses-cache
                        (cache-del-text-stat (misc-info->comm-to (car misclist))))
David Byers's avatar
X    
David Byers committed
1068
1069
1070
1071
1072
1073
                      (setq tmp
                            (cons
                             (blocking-do 'get-text-stat
                                          (misc-info->comm-to (car misclist)))
                             tmp)))
                     ((eq (misc-info->type (car misclist)) 'FOOTN-TO)
1074
1075
                      (unless kom-review-uses-cache
                        (cache-del-text-stat (misc-info->footn-to (car misclist))))
David Byers's avatar
X    
David Byers committed
1076
1077
1078
1079
1080
1081
                      (setq tmp 
                            (cons
                             (blocking-do 'get-text-stat
                                          (misc-info->footn-to (car misclist)))
                             tmp)))) 
               (setq misclist (cdr misclist)))
1082
1083
1084
1085
1086
1087
1088

	     ;;
	     ;; Remove unreadable texts
	     ;;

	     (setq tmp (delq nil tmp))

David Byers's avatar
X    
David Byers committed
1089
1090
1091
1092
1093
1094
1095
1096
1097
1098
1099
1100
1101
             ;;
             ;; If no parents were found, this is is a top-level text
             ;;

             (when (null tmp)
               (setq result (cons head result))
               (if (not all) (setq queue nil)))
             (setq queue (nconc tmp queue)))
           (if all (mapcar 'text-stat->text-no result)
             (text-stat->text-no (car result)))))
        (t nil)))


Per Cederqvist's avatar
.    
Per Cederqvist committed
1102
1103
1104
1105
1106


(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
1107
Text is a text-no."
Per Cederqvist's avatar
.    
Per Cederqvist committed
1108
1109
  (cond
   ((integerp text)
1110
1111
    (unless kom-review-uses-cache
      (cache-del-text-stat text))
1112
    (lyskom-view-text text nil t nil (lyskom-review-get-priority) t))
Linus Tolke's avatar
Linus Tolke committed
1113
1114
1115
1116
   (t
    (signal 'lyskom-internal-error
	    (list 'lyskom-review-tree
		  "Called with incorrect argument."
David Kågedal's avatar
David Kågedal committed
1117
		  text)))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1118
1119
1120
1121
1122
1123
1124
1125


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

;;; Author: Linus Tolke


David Byers's avatar
David Byers committed
1126
(def-kom-command kom-review-next ()
Per Cederqvist's avatar
.    
Per Cederqvist committed
1127
1128
1129
1130
  "Resumes an interupted review by moving all review and review-tree entries in
the lyskom-reading-list to the beginning. i.e by moving all other types to the
end."
  (interactive)
1131
  (lyskom-tell-internat 'kom-tell-review)
Per Cederqvist's avatar
.    
Per Cederqvist committed
1132
1133
1134
1135
1136
1137
1138
1139
1140
1141
  (let ((len (read-list-length lyskom-reading-list))
	(finished nil))
    (while (and (not finished)
		(> len 1))
      (let ((type (read-info->type (read-list->first lyskom-reading-list))))
	(if (and (not (eq type 'REVIEW))
		 (not (eq type 'REVIEW-TREE))
		 (not (eq type 'REVIEW-MARK)))
	    (read-list-rotate lyskom-reading-list)
	  (setq finished t)))
David Byers's avatar
David Byers committed
1142
      (setq len (1- len)))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1143
1144
1145
1146
1147
1148
1149
1150


;;; ================================================================
;;;                    ]terse stacken - Review stack

;;; Author: Linus


David Byers's avatar
David Byers committed
1151
(def-kom-command kom-review-stack ()
Per Cederqvist's avatar
.    
Per Cederqvist committed
1152
1153
  "Displays the review-stack."
  (interactive)
David Byers's avatar
David Byers committed
1154
1155
1156
1157
1158
1159
1160
1161
1162
1163
1164
1165
1166
1167
1168
1169
1170
1171
1172
1173
1174
1175
  (if (read-list->all-entries lyskom-reading-list)
      (mapcar
       (function
        (lambda (info)
          (let ((un (length (cdr (read-info->text-list info))))
                (type (read-info->type info))
                (cto (read-info->comm-to info))
                (conf (read-info->conf-stat info)))
            (cond
             ((eq type 'COMM-IN)
              (lyskom-format-insert 'view-many-comments cto un))
             ((eq type 'CONF)
              (lyskom-format-insert 'view-texts-in-conf un conf))
             ((eq type 'REVIEW)
              (lyskom-format-insert 'review-n-texts un))
             ((eq type 'REVIEW-TREE)
              (lyskom-format-insert 'review-many-comments cto un))
             ((eq type 'REVIEW-MARK)
              (lyskom-format-insert 'review-marked un))))))
       (read-list->all-entries lyskom-reading-list))
    (cond (lyskom-current-conf 
           (lyskom-format-insert 'you-have-no-unreads lyskom-current-conf))
David Byers's avatar
David Byers committed
1176
          (t (lyskom-insert 'not-reading-anywhere)))))
David Byers's avatar
David Byers committed
1177

Per Cederqvist's avatar
.    
Per Cederqvist committed
1178
1179
1180
1181
1182
1183
1184
1185


;;; ================================================================
;;;                      ]terse hoppa - review clear

;;; Author: Linus Tolke


David Byers's avatar
David Byers committed
1186
(def-kom-command kom-review-clear ()
Per Cederqvist's avatar
.    
Per Cederqvist committed
1187
1188
  "Deletes all review-types from the lyskom-reading-list and lyskom-to-do-list."
  (interactive)
David Byers's avatar
David Byers committed
1189
1190
  (let ((found nil))
    (if (not (read-list-isempty lyskom-reading-list))
1191
1192
        (while (memq (read-info->type (read-list->first lyskom-reading-list))
                       lyskom-review-types-list)
David Byers's avatar
David Byers committed
1193
1194
1195
          (set-read-list-del-first lyskom-reading-list)
          (setq found t)))
    (if (not (read-list-isempty lyskom-to-do-list))
1196
1197
        (while (memq (read-info->type (read-list->first lyskom-to-do-list))
                       lyskom-review-types-list)
David Byers's avatar
David Byers committed
1198
1199
1200
1201
          (set-read-list-del-first lyskom-to-do-list)
          (setq found t)))

    (unless found (lyskom-insert 'not-reviewing))))
Per Cederqvist's avatar
.    
Per Cederqvist committed
1202
1203
1204
1205
1206
1207
1208
1209


;;; ================================================================
;;;          ]terse det kommenterade - View commented text

;;; Author: Inge Wallin


1210
(def-kom-command kom-review-comments (text-no)
Per Cederqvist's avatar
.    
Per Cederqvist committed
1211
1212
1213
  "View the comments to this text.
If the current text has comments in (footnotes in) some texts then the first
text is shown and a REVIEW list is built to shown the other ones."
David Byers's avatar
David Byers committed
1214
  (interactive (list (lyskom-read-text-no-prefix-arg 'review-comments-q)))
1215
  (lyskom-tell-internat 'kom-tell-review)
1216
  (cond (text-no
1217
1218
         (unless kom-review-uses-cache
           (cache-del-text-stat text-no))
1219
1220
         (lyskom-review-comments
          (blocking-do 'get-text-stat text-no)))
David Byers's avatar
David Byers committed
1221
        (t lyskom-insert-string 'read-text-first)))
1222
1223
1224
1225
1226


(defun lyskom-review-comments (text-stat)
  "Handles the return from the initiate-get-text-stat, displays and builds list."
  (let* ((misc-info-list (and text-stat
Per Cederqvist's avatar
.    
Per Cederqvist committed
1227
1228
1229
			      (text-stat->misc-info-list text-stat)))
	 (misc-infos (and misc-info-list
			  (append (lyskom-misc-infos-from-list 
David Kågedal's avatar
David Kågedal committed
1230
				   'FOOTN-IN misc-info-list)
Per Cederqvist's avatar
.    
Per Cederqvist committed
1231
				  (lyskom-misc-infos-from-list 
David Kågedal's avatar
David Kågedal committed
1232
				   'COMM-IN misc-info-list))))
1233
	 (all-text-nos (and misc-infos
Per Cederqvist's avatar
.    
Per Cederqvist committed
1234
1235
1236
1237
1238
1239
1240
			(mapcar
			 (function
			  (lambda (misc-info)
			    (if (equal (misc-info->type misc-info)
				       'COMM-IN)
				(misc-info->comm-in misc-info)
			      (misc-info->footn-in misc-info))))
1241
1242
1243
1244
			 misc-infos)))
         text-nos)
    ;; Only try to review texts that we can read.
    (while all-text-nos
1245
1246
      (unless kom-review-uses-cache
        (cache-del-text-stat (car all-text-nos)))
1247
1248
1249
      (if (blocking-do 'get-text-stat (car all-text-nos))
          (setq text-nos (cons (car all-text-nos) text-nos)))
      (setq all-text-nos (cdr all-text-nos)))
Joel Rosdahl's avatar
Joel Rosdahl committed
1250
    (setq text-nos (nreverse text-nos))
1251

Per Cederqvist's avatar
.    
Per Cederqvist committed
1252
1253
1254
1255
    (if text-nos
	(progn
	  (lyskom-format-insert 'review-text-no (car text-nos))
	  (if (cdr text-nos)
1256
1257
1258
1259
1260
              (lyskom-review-enter-read-info
               (lyskom-create-read-info
                'REVIEW nil (lyskom-review-get-priority)
                (lyskom-create-text-list (cdr text-nos))
                lyskom-current-text) t))
1261
1262
1263
          ;; Don't check the no-cache thing here since we already
          ;; did earlier. We may end up slightly out of sync with
          ;; the server, but not so anyone will really notice.
1264
	  (lyskom-view-text (car text-nos)))
1265
      (lyskom-insert-string 'no-such-text))))
1266
1267
1268
1269
1270
1271
1272
1273


;;; ================================================================
;;;          ]terse igen - kom-review-last-normally-read
;;;
;;; Author: Linus Tolke


David Byers's avatar
David Byers committed
1274
(def-kom-command kom-review-last-normally-read (no)
1275
1276
1277
1278
  "Reviews the NO last normally read texts."
  (interactive 
   (list 
    (lyskom-read-number (lyskom-get-string 'read-normally-read) 1)))
1279
  (lyskom-tell-internat 'kom-tell-review)
1280
  (let* ((text-nos (nreverse (nfirst no lyskom-normally-read-texts))))
1281
1282
1283
1284
    (if text-nos
	(progn
	  (lyskom-format-insert 'review-text-no (car text-nos))
	  (if (cdr text-nos)
1285
1286
1287
1288
1289
	      (lyskom-review-enter-read-info
               (lyskom-create-read-info
                'REVIEW nil (lyskom-review-get-priority)
                (lyskom-create-text-list (cdr text-nos))
                lyskom-current-text) t))
1290
1291
          (unless kom-review-uses-cache
            (cache-del-text-stat (car text-nos)))
1292
	  (lyskom-view-text (car text-nos)))
David Byers's avatar
David Byers committed
1293
      (lyskom-format-insert 'no-such-text))))
David Kågedal's avatar
David Kågedal committed
1294
1295


1296
1297
1298
1299
1300
;;; ============================================================
;;;        Återse omodifierat - Review a non-converted text

;;; Author: Linus Tolke
;;; Modified by: Johan Sundström
David Kågedal's avatar
David Kågedal committed
1301

1302
(def-kom-command kom-review-noconversion (text-no)
1303
1304
  "Displays TEXT-NO or the last read text without any conversion."
  (interactive (list (lyskom-read-text-no-prefix-arg 'review-noconversion-q)))
1305
1306
1307
1308
  (if text-no
      (let ((lyskom-format-special nil)
            (kom-smileys nil)
            (kom-autowrap nil))
1309
1310
          (unless kom-review-uses-cache
            (cache-del-text-stat text-no))
1311
1312
        (lyskom-view-text text-no))
    (lyskom-insert 'confusion-what-to-view)))
David Kågedal's avatar
David Kågedal committed
1313
1314
1315



1316
1317
1318
1319
1320
1321
1322
1323
1324
(defun lyskom-review-get-priority ()
  "Get the priority to use for reviewing texts."
  (or kom-review-priority (lyskom-get-current-priority)))

(defun lyskom-review-enter-read-info (read-info before)
  "Enter READ-INFO into lyskom-reading-list and lyskom-to-do-list."
  (read-list-enter-read-info read-info lyskom-reading-list before)
  (read-list-enter-read-info read-info lyskom-to-do-list before))

David Byers's avatar
David Byers committed
1325
1326
1327
;;; ============================================================
;;;         Återse senaste dagarnas inlägg
;;;
1328
;;; Author: Up for grabs
David Byers's avatar
David Byers committed
1329
1330
1331
1332
1333
1334
1335
1336
1337
1338
1339
1340
1341
1342
1343
1344
1345
1346
1347
1348
1349
1350

;;;
;;; Algorithm:
;;;
;;; Binärsökning i mappen efter inlägg med rätt datum.
;;;
;;; Utilityfunktioner: Beräkna diff i dagar mellan två datum.
;;;                    Subtrahera n dagar från ett datum.
;;;
;;; Användarkommandot: Återse N inlägg av person X till Y från DATE
;;;                    
;;; Hitta index i X och Y där det sökta datumet börjar
;;; Låt de vanliga återsefunktionerna accepter cmin och pmin som
;;; parametrar så de kan söka igenom en del av en map.
;;;
;;; Problem med binärsökning i mapparna är hålen. Det finns risk att
;;; man försöker binärsöka bland en massa nollor, och det lär ta tid.
;;; I de fallen får man nog göra get-next-text eller get-previous-text
;;; för att få ett riktigt textnummer att titta på. Risk: att man
;;; tittar på samma text två gånger och går i loop.
;;;