Semantics: appWasMade

[kangarang]

I think we should reword function appWasMade to isInApplication or isBeingReviewed or something else to better describe the stage/timing of the domain's current state. Since items in listingMap are deleted during resetListing, it could be confusing if appWasMade correctly returns false for an application that i definitely made, but is completed and already whitelisted.

[miguelmota]

agreed, isInApplication is better imo

[skmgoldin]

Just so we have it here to look at:

AdChainRegistry/contracts/Registry.sol

Lines 306 to 309 in cf0d0f5

	// Returns true if apply(domain) was called for this domain
	function appWasMade(string _domain) constant public returns (bool exists) {
	return listingMap[keccak256(_domain)].applicationExpiry > 0;
	}

[kangarang]

Actually, I take this back. items in listingMap only get deleted by resetListing if and only if:

• a challenge succeeds
• the applicant invokes exit
• the listing struct's unstakedDeposit is for some reason less that the minimumDeposit

So appWasMade returns true (as expected) for white-listed applications.

My apologies for this misunderstanding. I was definitely confused.

[skmgoldin]

If a domain applies and either is or is not listed, we don't care, then it is true forever that that domain did apply. Semantically. So maybe this needs to be named domainIsInApplicationOrIsListed?

[kangarang]

domainIsActive
domainWasInitialized

[skmgoldin]

I'm not afraid of long method names.

[kangarang]

Right on! Let's go with domainIsInApplicationOrIsListed then since it's the most specific.