logo       

[HtmlUnit] [ htmlunit-Bugs-1419050 ] HtmlElement attributes whithout order: msg#00004

java.htmlunit.devel

Subject: [HtmlUnit] [ htmlunit-Bugs-1419050 ] HtmlElement attributes whithout order

Bugs item #1419050, was opened at 2006-01-30 21:32
Message generated for change (Settings changed) made by mguillem
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=448266&aid=1419050&group_id=47038

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: Latest code in CVS
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Marcos Vinicius (marvinsouza)
>Assigned to: Marc Guillemot (mguillem)
Summary: HtmlElement attributes whithout order

Initial Comment:
English:
The attributes of a HtmlElement don't have none order.
In the class DefaultElementFactory they are placed in a
HashMap, therefore, in the moment of iterate, they are
disordered.
The solution would be placed them in a TreeMap.
Attached, folow the possible correction.
Sorry my poor english :)

Portuguese:
Os atributos de um HtmlElement não possuem nenhuma
ordenação. Na classe DefaultElementFactory eles são
colocados em um HashMap, por isso, na hora de
percorrê-los eles não respeitam nenhuma ordem.
A solução seria colocá-los em um TreeMap.
Em anexo, segue a possível correção.


----------------------------------------------------------------------

>Comment By: Marc Guillemot (mguillem)
Date: 2006-02-28 19:03

Message:
Logged In: YES
user_id=402164

Now fixed in CVS.

Note: there is no reason to place the attributes in
attribute name lexical order. There are now returned in the
order in which they are found in the html source code.

----------------------------------------------------------------------

Comment By: Marc Guillemot (mguillem)
Date: 2006-01-31 08:58

Message:
Logged In: YES
user_id=402164

Your patch is welcome but please follow the submission
guidelines
http://htmlunit.sourceforge.net/submittingPatches.html and
provide the required unit tests.


----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=448266&aid=1419050&group_id=47038


-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642


<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

News | FAQ | advertise