Rewrite title question don't add if

[pkjack]

@kristian re: Rewriting the title - #32 by kristian

Hiya!! so the code has been working great! so thanks for that… but, I do have a question though… is there away of adding this…

e.g if “Pattern” attribute is “Floral” don’t add it to the title?

Hope that makes sense?

All the best

[kristian]

@pkjack

e.g if “Pattern” attribute is “Floral” don’t add it to the title?

This seems to be possible by just adding
|| listing.attributes["Pattern"]=="Floral"

This is the new script:

function run() {

	for (const listing of selectedListings) {
		
		var thisBrand = "";
		var thisSize = "";
		var thisColour = "";
		var thisPattern = "";
		var thisFeatures = "";
		var thisType = "";

		if (!listing.attributes["Brand"] || listing.attributes["Brand"]=="") {
			consoleLog("NO BRAND");
		} else {
			thisBrand = listing.attributes["Brand"] + " ";
		}

		if (!listing.attributes["Size"] || listing.attributes["Size"]=="") {
			consoleLog("NO SIZE");
		} else {
			thisSize = "Size" + " " + listing.attributes["Size"] + " ";
		}

		if (!listing.attributes["Colour"] || listing.attributes["Colour"]=="") {
			consoleLog("NO COLOUR");
		} else {
			thisColour = listing.attributes["Colour"] + " ";
		}

		if (!listing.attributes["Pattern"] || listing.attributes["Pattern"]=="" || listing.attributes["Pattern"]=="Floral") {
			consoleLog("NO PATTERN");
		} else {
			thisPattern = listing.attributes["Pattern"] + " ";
		}

		if (!listing.attributes["Features"] || listing.attributes["Features"]=="") {
			consoleLog("NO FEATURES");
		} else {
			thisFeatures = listing.attributes["Features"] + " ";
		}

		if (!listing.attributes["Type"] || listing.attributes["Type"]=="") {
			consoleLog("NO TYPE");
		} else {
			thisType = listing.attributes["Type"] + " ";
		}

		listing.title = "Text" + " " + thisBrand + thisSize + thisColour + thisPattern + thisFeatures + thisType + "Text";

	}
}

[pkjack]

Thats perfect!!! thanks again!

[pkjack]

Hey Kristian. 1 last question on this… Is there anyway to use an ‘if’ - with the same example… if pattern is floral then replace with “flowers” - again only the title… not actual attribute

All the best

[kristian]

@pkjack

if pattern is floral then replace with “flowers” - again only the title… not actual attribute

This should do the trick:

function run() {

	for (const listing of selectedListings) {
		
		var thisBrand = "";
		var thisSize = "";
		var thisColour = "";
		var thisPattern = "";
		var thisFeatures = "";
		var thisType = "";

		if (!listing.attributes["Brand"] || listing.attributes["Brand"]=="") {
			consoleLog("NO BRAND");
		} else {
			thisBrand = listing.attributes["Brand"] + " ";
		}

		if (!listing.attributes["Size"] || listing.attributes["Size"]=="") {
			consoleLog("NO SIZE");
		} else {
			thisSize = "Size" + " " + listing.attributes["Size"] + " ";
		}

		if (!listing.attributes["Colour"] || listing.attributes["Colour"]=="") {
			consoleLog("NO COLOUR");
		} else {
			thisColour = listing.attributes["Colour"] + " ";
		}

		if (!listing.attributes["Pattern"] || listing.attributes["Pattern"]=="") {
			consoleLog("NO PATTERN");
		} else {
			if (listing.attributes["Pattern"]=="Floral") {
				thisPattern = "Flower ";
			} else {
			 thisPattern = listing.attributes["Pattern"] + " ";
			}
		}

		if (!listing.attributes["Features"] || listing.attributes["Features"]=="") {
			consoleLog("NO FEATURES");
		} else {
			thisFeatures = listing.attributes["Features"] + " ";
		}

		if (!listing.attributes["Type"] || listing.attributes["Type"]=="") {
			consoleLog("NO TYPE");
		} else {
			thisType = listing.attributes["Type"] + " ";
		}

		listing.title = "Text" + " " + thisBrand + thisSize + thisColour + thisPattern + thisFeatures + thisType + "Text";

	}
}

Regards, Kristian