Revision: 3547
Author: ksprotte
URL: http://bknr.net/trac/changeset/3547
fixed bug reported by vivian.
"fehler bei ueberweisung ohne email
Mir ist im Laufe der Tage folgendes aufgefallen:
wenn ueberweiser keine emailadresse angeben und dann ueber den link
aktiviert werden kommt eine Fehlermeldung die besag could not be
proceeded, must contain domain bla....
Der Sponsor wird aber dann trotzdem als bezahlt eingerichtet und sein
Profil erstellt. Diese Fehlermeldung muss heissen:
"Sponsor processed, but not informed because no mail address" oder so was....."
U trunk/projects/bos/m2/mail-generator.lisp
Modified: trunk/projects/bos/m2/mail-generator.lisp
===================================================================
--- trunk/projects/bos/m2/mail-generator.lisp 2008-07-22 08:56:46 UTC (rev 3546)
+++ trunk/projects/bos/m2/mail-generator.lisp 2008-07-22 09:35:08 UTC (rev 3547)
@@ -327,7 +327,7 @@
vorname name strasse plz ort email telefon
(if want-print "yes" "no")
(if donationcert-yearly "yes" "no")
- *website-url* contract-id email))
+ *website-url* contract-id (or email "")))
(make-contract-xml-part contract-id (all-request-params))
(make-vcard-part contract-id (make-vcard :sponsor-id sponsor-id
:note (format nil "Paid-by: Manual money transfer
Revision: 3546
Author: ksprotte
URL: http://bknr.net/trac/changeset/3546
small fix for an assertion in make-sat-layer
highest local-draw-order is reserved for the contract-tree, so
make-sat-layer checks that this last level is not used for the
sat-layer
U trunk/projects/bos/web/sat-tree.lisp
Modified: trunk/projects/bos/web/sat-tree.lisp
===================================================================
--- trunk/projects/bos/web/sat-tree.lisp 2008-07-22 08:50:30 UTC (rev 3545)
+++ trunk/projects/bos/web/sat-tree.lisp 2008-07-22 08:56:46 UTC (rev 3546)
@@ -185,7 +185,8 @@
(assert (geo-box-encloses-p *m2-geo-box* geo-box))
(check-type start-depth (integer 0))
(check-type local-draw-order (integer 0))
- (assert (< local-draw-order +max-num-of-local-draw-order-levels+))
+ ;; highest local-draw-order is reserved for the contract-tree
+ (assert (< local-draw-order (1- +max-num-of-local-draw-order-levels+)))
(when (find local-draw-order (class-instances 'sat-layer) :key #'local-draw-order)
(cerror "create the new layer anyway" "There is already a sat-layer with the same local-draw-order '~A'." local-draw-order))
(let ((layer (make-object 'sat-layer :name name :geo-box geo-box :local-draw-order local-draw-order)))
Revision: 3542
Author: ksprotte
URL: http://bknr.net/trac/changeset/3542
removed (bos.m2.allocation-cache:rebuild-cache) from daily job, cause it is done in snapshot (and wrong to do there anyway - because rebuild-cache is not a transaction, but would need to be logged)
U trunk/projects/bos/m2/allocation-cache.lisp
U trunk/projects/bos/web/daily.lisp
Modified: trunk/projects/bos/m2/allocation-cache.lisp
===================================================================
--- trunk/projects/bos/m2/allocation-cache.lisp 2008-07-22 05:40:55 UTC (rev 3541)
+++ trunk/projects/bos/m2/allocation-cache.lisp 2008-07-22 08:32:05 UTC (rev 3542)
@@ -232,6 +232,7 @@
(format t "~5D~10T~5D~%" size count))))))
(defun rebuild-cache ()
+ ""
(unless *allocation-cache*
(setq *allocation-cache* (make-allocation-cache)))
(clear-cache)
Modified: trunk/projects/bos/web/daily.lisp
===================================================================
--- trunk/projects/bos/web/daily.lisp 2008-07-22 05:40:55 UTC (rev 3541)
+++ trunk/projects/bos/web/daily.lisp 2008-07-22 08:32:05 UTC (rev 3542)
@@ -7,8 +7,6 @@
(bknr.stats::make-yesterdays-stats :delete-events t)
(format t "; snapshotting datastore~%")
(snapshot)
- (format t "; rebuilding allocation-cache~%")
- (bos.m2.allocation-cache:rebuild-cache)
(format t "; running check (consistent-p)~%")
(flet ((consistent-p-report ()
(let (consistent-p)