ytiddo,
I’m confused by this:
“if you leave the record number empty, a url is generated incorrectly. this fixes the url generation.”
How do I leave a record number empty? If you want a review, we need to be able to recreate the error.
Plus, why am I spending the time figuring out and testing something that has already likely been appropriately tested?
the problem occurs when the first item in the page that is being dupe checked is empty, and optional.
the external ID field is what i was referring to.
As for why you’re reviewing… As I work with steven, i figured you would NOT want us reviewing one anothers patches and comitting, as this could cause workplace pressures (of which we have plenty) to short circuit the review process.