Hello,

Nowadays the DayMarker builder has a gui() which returns the integer
value stored internaly.
Wouldnt it be more logical to return the actual day which is refered to
here?

And no, im not gonna ask for a vote, only when people think it is really
required.

Index: DayMarkers.java
===================================================================
RCS file: /usr/local/cvs/mmbase/module/builders/DayMarkers.java,v
retrieving revision 1.23
diff -r1.23 DayMarkers.java
32a33,34
>     
>     public static String FIELD_DAYCOUNT =   "daycount";
446a449,470
> 
>     
>     /**
>      *  Returns the date of a daymarker
>      *  @param node The node of which the date is wanted
>      *  @return a <code>Date</code> which is the date
>      */
>     public java.util.Date getDate(MMObjectNode node) {
>         int dayCount = node.getIntValue(FIELD_DAYCOUNT);
>          return new
java.util.Date(((long)dayCount)*24*3600*1000);        
>     }
>     
>     /**
>      *  Returns gui information for a specific node. This value is
retrieved by retrieving the field 'gui()' of the node
(node.getStringValue("gui()") )
>      *  @param node The node of which the gui information is wanted
>      *  @return a <code>String</code> in which the current date is
shown
>      */
>     public String getGUIIndicator(MMObjectNode node) {    
>         java.text.DateFormat format =
java.text.DateFormat.getDateInstance(java.text.DateFormat.DEFAULT,
java.util.Locale.getDefault());
>         return format.format(getDate(node));
>     }
> 
-- 
Eduard Witteveen Systeem Ontwikkelaar
NOS Internet,  Gateway C Kamer 107
+31(0)356772910 http://www.omroep.nl/

Sed quis custodiet ipsos custodes? : The sixth Satire from Juvenal

Reply via email to