[Building Sakai] K1 - uitils rationalization?

[David Horwitz]

Thanks guys - will wait for those in GMT- to have a chance to repond
before I go ahead. In the meantime I have created a jira to track this:

http://jira.sakaiproject.org/jira/browse/KNL-204

D

DAVID ROLDAN MARTINEZ wrote:
> +1
>
> De: sakai-dev-bounces at collab.sakaiproject.org [mailto:sakai-dev-bounces at collab.sakaiproject.org] En nombre de Nuno Fernandes
> Enviado el: viernes, 05 de junio de 2009 11:35
> Para: Aaron Zeckoski
> CC: sakai-dev
> Asunto: Re: [Building Sakai] K1 - uitils rationalization?
>
> +1
> 2009/6/5 Aaron Zeckoski <aaronz at vt.edu<mailto:aaronz at vt.edu>>
> Good idea.
> +1
> -AZ
>
> On Fri, Jun 5, 2009 at 10:05 AM, David Horwitz<david.horwitz at uct.ac.za<mailto:david.horwitz at uct.ac.za>> wrote:
>   
>> Hi All,
>>
>> I was looking at the utils classes in the K1 kernel and it strikes me
>> that we could do with some rationalization in this area to achieve:
>>
>> 1) Remove duplicated functionality
>> 2) Remove custom Sakai code that has the same functionality as standard
>> java classes (specifically Apache commons libraries)
>>
>>
>> Some of the ones I noticed:
>>
>>
>> 1) EmptyIterator - present in commons-collections
>> 2) Web.escapeJsQuotes & Validator.escapeJsQuoted() - duplicate code, in
>> commons-lang StringEscapeUtils, one marked deprecated and buggy (KNL-69)
>> 3) Validator.escapeSql in commons-lang StringEscapeUtils
>> 4) Validator.esapceJavascript in commons-lang StringEscapeUtils
>> 5) Validator.checkEmailLocal in commons-validator
>> 6) StringUtils.split - "Like jdk1.4's String.split..." the javadoc says
>> it all this is now in the jvm
>> 7) StringUtils.trimToNull & StringUtils.trimToEmpty
>>
>> This is just a quick survey and I'm sure there more. I Propose that we
>> start by marking all these as deprecated and point people the the
>> commons implementation. Thoughts?
>>
>>
>> David
>> _______________________________________________
>> sakai-dev mailing list
>> sakai-dev at collab.sakaiproject.org<mailto:sakai-dev at collab.sakaiproject.org>
>> http://collab.sakaiproject.org/mailman/listinfo/sakai-dev
>>
>> TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org<mailto:sakai-dev-unsubscribe at collab.sakaiproject.org> with a subject of "unsubscribe"
>>
>>     
>
>
> --
> Aaron Zeckoski (azeckoski (at) vt.edu<http://vt.edu>)
> Senior Research Engineer - CARET - Cambridge University
> https://twitter.com/azeckoski - http://www.linkedin.com/in/azeckoski
> http://aaronz-sakai.blogspot.com/ -
> http://confluence.sakaiproject.org/confluence/display/~aaronz/
> _______________________________________________
> sakai-dev mailing list
> sakai-dev at collab.sakaiproject.org<mailto:sakai-dev at collab.sakaiproject.org>
> http://collab.sakaiproject.org/mailman/listinfo/sakai-dev
>
> TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org<mailto:sakai-dev-unsubscribe at collab.sakaiproject.org> with a subject of "unsubscribe"
>
>
>
> --
> Nuno Fernandes  [www.linkedin.com/in/nfgrilo<http://www.linkedin.com/in/nfgrilo>]
> Software Developer - UFP-UV  [Universidade Virtual]
> Software Developer - Sakai Foundation  [sakaiproject.org<http://sakaiproject.org>]
> Sakai Fellow 2008 - Sakai Foundation  [confluence.sakaiproject.org/confluence/x/FIF9AQ<http://confluence.sakaiproject.org/confluence/x/FIF9AQ>]
>
> Universidade Fernando Pessoa  [www.ufp.pt<http://www.ufp.pt>]
> Praça 9 de Abril, 349
> 4249-004 Porto
> tel: + 351 22 507 13 00
> fax: + 351 22 550 82 69
>
>   
> ------------------------------------------------------------------------
>
> _______________________________________________
> sakai-dev mailing list
> sakai-dev at collab.sakaiproject.org
> http://collab.sakaiproject.org/mailman/listinfo/sakai-dev
>
> TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org with a subject of "unsubscribe"
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://collab.sakaiproject.org/pipermail/sakai-dev/attachments/20090605/b9ebb4f1/attachment.html