Closed
Bug 412660
Opened 18 years ago
Closed 17 years ago
Need to better handle case where an adr contains only an extended-address
Categories
(Toolkit Graveyard :: Microformats, defect)
Toolkit Graveyard
Microformats
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mkaply, Assigned: mkaply)
Details
Attachments
(1 file, 1 obsolete file)
|
4.50 KB,
patch
|
sayrer
:
review+
mtschrep
:
approval1.9+
|
Details | Diff | Splinter Review |
In the microformats code, I attempt to create a title for a given microformat.
In the adr case, I do not use "extended-address" at all, but there are cases now where extended-address is the only thing specified.
|
Assignee | |
Comment 1•18 years ago
|
||
If there is an extended-address, use it when creating the label of the microformat.
Attachment #297393 -
Flags: review?
|
|
Assignee | |
Updated•18 years ago
|
Attachment #297393 -
Flags: review? → review?(sayrer)
|
||
Comment 2•18 years ago
|
||
Needs a test.
|
|
Assignee | |
Comment 3•18 years ago
|
||
Fix that includes unit test to cover this case in adr
Attachment #297393 -
Attachment is obsolete: true
Attachment #301511 -
Flags: review?(sayrer)
Attachment #297393 -
Flags: review?(sayrer)
|
|
||
Updated•17 years ago
|
Attachment #301511 -
Flags: review?(sayrer) → review+
|
|
Assignee | |
Updated•17 years ago
|
Attachment #301511 -
Flags: approval1.9?
|
||
Comment 4•17 years ago
|
||
Comment on attachment 301511 [details] [diff] [review]
Fix for problem with test
Thnx for the tests.
Attachment #301511 -
Flags: approval1.9? → approval1.9+
|
|
Assignee | |
Comment 5•17 years ago
|
||
Checked in.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
|
||
Updated•6 years ago
|
Product: Toolkit → Toolkit Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•