"you gives you a merch" message after delivering food to free merchant's broker

[gettingusedto]

Describe the bug

The popup text that appears after delivering food to the broker says you gives you x merch instead of <npc_name> gives you x merch.

Attach save file

N/A

Steps to reproduce

1. get yourself an item the broker can receive, like fruit wine and go to the refugee center,
2. give the item to the broker through the dialogue,
3. "you gives you a merch".

Expected behavior

The broker being the one that gives the money, not me going to their wallet and take a merch or two, that'd be rude.

Screenshots

Versions and configuration

• OS: Android
  • OS Version: Manufacturer: samsung; Model: SM-A710F; Release: 7.0; Incremental: A710FXXU2CRC2;
• Game Version: 2f71c08 [32-bit]
• Graphics Version: Tiles
• Game Language: English [en]
• Mods loaded: [
  Dark Days Ahead [dda],
  Disable NPC Needs [no_npc_food],
  No Fungal Growth [no_fungal_growth],
  Portal Storms Ignore NPCs [personal_portal_storms],
  No scent tracking zombies [no_scent_tracking_zombies],
  Magiclysm [magiclysm],
  Xedra Evolved [xedra_evolved],
  Misc Sprites [misc_sprite]
  ]

Additional context

No response

[GuardianDll]

Someone need to slap a new bool from #64070 into the broker json

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not bump or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered.

[Procyonae]

/confirm alef got this in a normal play session

[SurFlurer]

Cataclysm-DDA/src/npctalk.cpp

Lines 3444 to 3456 in 49c08f4

	if( buyer_has == 1 ) {
	//~ %1%s is the NPC name, %2$s is an item
	popup( _( "%1$s gives you a %2$s." ), seller->disp_name(),
	pay.tname() );
	} else if( buyer_has > 1 ) {
	//~ %1%s is the NPC name, %2$d is a number of items, %3$s are items
	popup( _( "%1$s gives you %2$d %3$s." ), seller->disp_name(), buyer_has,
	pay.tname() );
	}
	}
	for( int i = 0; i < buyer_has; i++ ) {
	seller->i_add( pay );
	price -= value;

It's almost certain that it should be the buyer that gives currency to the seller 😄