🕸Fergus Duniho wrote on Tue, Mar 1, 2022 05:35 PM UTC:
I was merging the functionality of canonicalURI, which takes an ItemID, into make_link_url, which takes a row, and I didn't do a thorough job of it. That is now fixed. Instead of duplicating functionality, canonicalURI will now use the ItemID to get a row with which it can call make_link_url. Having make_link_url call canonicalURI, as it was doing before, was wasteful, because it queried the database for a row it already had from a previous database query.
I was merging the functionality of canonicalURI, which takes an ItemID, into make_link_url, which takes a row, and I didn't do a thorough job of it. That is now fixed. Instead of duplicating functionality, canonicalURI will now use the ItemID to get a row with which it can call make_link_url. Having make_link_url call canonicalURI, as it was doing before, was wasteful, because it queried the database for a row it already had from a previous database query.