gmazza 2004/12/01 17:59:41
Modified: src/java/org/apache/fop/area AreaTreeHandler.java
LinkResolver.java Resolvable.java
src/java/org/apache/fop/area/inline
UnresolvedPageNumber.java
Log:
More commenting, changed definition of isResolved() somewhat in LinkResolver
and UnresolvedPageNumber.
Revision Changes Path
1.21 +11 -7 xml-fop/src/java/org/apache/fop/area/AreaTreeHandler.java
Index: AreaTreeHandler.java
===================================================================
RCS file:
/home/cvs/xml-fop/src/java/org/apache/fop/area/AreaTreeHandler.java,v
retrieving revision 1.20
retrieving revision 1.21
diff -u -r1.20 -r1.21
--- AreaTreeHandler.java 2 Dec 2004 01:36:09 -0000 1.20
+++ AreaTreeHandler.java 2 Dec 2004 01:59:40 -0000 1.21
@@ -160,7 +160,7 @@
/**
* Add an Resolvable object with an unresolved idref
* @param idref the idref whose target id has not yet been located
- * @param res the Resolvable object with the unresolved idref
+ * @param res the Resolvable object needing the idref to be resolved
*/
public void addUnresolvedIDRef(String idref, Resolvable res) {
Set todo = (Set) unresolvedIDRefs.get(idref);
@@ -193,15 +193,19 @@
* @throws SAXException if there is some error
*/
public void endDocument() throws SAXException {
- // deal with unresolved references
+ /*
+ * inform Resolveable objects that certain idrefs
+ * could not be found
+ * @todo unsure if this block is needed.
+ */
for (Iterator iter = unresolvedIDRefs.keySet().iterator();
iter.hasNext();) {
- String id = (String)iter.next();
- Set list = (Set) unresolvedIDRefs.get(id);
+ String idref = (String) iter.next();
+ Set list = (Set) unresolvedIDRefs.get(idref);
for (Iterator resIter = list.iterator(); resIter.hasNext();) {
- Resolvable res = (Resolvable)resIter.next();
+ Resolvable res = (Resolvable) resIter.next();
if (!res.isResolved()) {
- res.resolveIDRef(id, null);
+ res.resolveIDRef(idref, null);
}
}
}
1.5 +2 -2 xml-fop/src/java/org/apache/fop/area/LinkResolver.java
Index: LinkResolver.java
===================================================================
RCS file: /home/cvs/xml-fop/src/java/org/apache/fop/area/LinkResolver.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -r1.4 -r1.5
--- LinkResolver.java 2 Dec 2004 01:36:09 -0000 1.4
+++ LinkResolver.java 2 Dec 2004 01:59:40 -0000 1.5
@@ -62,8 +62,8 @@
* Resolve by adding an internal link.
*/
public void resolveIDRef(String id, List pages) {
- resolved = true;
if (idRef.equals(id) && pages != null) {
+ resolved = true;
PageViewport page = (PageViewport)pages.get(0);
area.addTrait(Trait.INTERNAL_LINK, page.getKey());
}
1.4 +9 -5 xml-fop/src/java/org/apache/fop/area/Resolvable.java
Index: Resolvable.java
===================================================================
RCS file: /home/cvs/xml-fop/src/java/org/apache/fop/area/Resolvable.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- Resolvable.java 2 Dec 2004 01:36:09 -0000 1.3
+++ Resolvable.java 2 Dec 2004 01:59:40 -0000 1.4
@@ -45,11 +45,15 @@
String[] getIDs();
/**
- * This method provides the opportunity for a Resolvable object
- * to resolve one of its unresolved idrefs with the actual set of
- * PageViewports containing the target ID.
+ * This method allows the Resolvable object to resolve one of
+ * its unresolved idrefs with the actual set of PageViewports
+ * containing the target ID. The Resolvable object initially
+ * identifies to the AreaTreeHandler which idrefs it needs
+ * resolved. After the idrefs are resolved, the ATH calls this
+ * method to allow the Resolvable object to update itself with
+ * the PageViewport information.
*
- * @param id an ID possibly matching one of the Resolvable object's
+ * @param id an ID matching one of the Resolvable object's
* unresolved idref's.
* @param pages the list of PageViewports with the given ID
* may be null if ID is not tied to any
1.6 +1 -4
xml-fop/src/java/org/apache/fop/area/inline/UnresolvedPageNumber.java
Index: UnresolvedPageNumber.java
===================================================================
RCS file:
/home/cvs/xml-fop/src/java/org/apache/fop/area/inline/UnresolvedPageNumber.java,v
retrieving revision 1.5
retrieving revision 1.6
diff -u -r1.5 -r1.6
--- UnresolvedPageNumber.java 2 Dec 2004 01:36:09 -0000 1.5
+++ UnresolvedPageNumber.java 2 Dec 2004 01:59:40 -0000 1.6
@@ -60,16 +60,13 @@
*
* @param id the id associated with the pages parameter
* @param pages the list of PageViewports associated with this ID
- * @todo determine why the ID passed in will (always?) equal the
pageIDRef,
- * explain in comments above
*/
public void resolveIDRef(String id, List pages) {
- resolved = true;
if (pages != null) {
+ resolved = true;
PageViewport page = (PageViewport)pages.get(0);
String str = page.getPageNumber();
text = str;
-
/[EMAIL PROTECTED] Update IPD ??? */
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]