Had a quick test (going through all the supported pages, and checking data is collected / that theres no change logged when swapping between viewing a property details/in a search list)
Looks good, and couldn't find any problems
Tho noticed (and I didn't support this first time round and no one has asked for it, so mentioning it only for completeness) that propertynews do the same as daft.ie and have a page for a group of development properties (new homes & overseas), like
http://www.propertynews.com/development ... p=1206&r=& . These can be awkward to support and of limited use I feel, so I'm suggesting not supporting these pages!
Re Agents address, I'm not sure how important it is, only real uses are possibly
* seeing if properties have been moved between EA offices
* being able to find all properties belonging to a particular EA office
Neither of which is available via the toolbar (tho could do this with a bit of sql on property-bee.sqlite or exporting the data into a spreadsheet - for those that are keen!)
If you want to add in the agents address, I'd use PB_AllNodesMatchingXPath() to iterate over the <li> elements which will concatenate them into 1 string with each element separated by a comma - so doesn't matter how many lines there are. Also I think the defaultValue should be '' which would mean that if not found, it will ignore the address/change.
Cheers
BH
)
