Planet for Surface signals detected

[CMDR-Evolution]

Hello EDDI,

When we hear about surface signals, I want it to say the planet (which is in the journal file) but If there are two planets in range, it tells me that both sets are on the planet I'm landed on.
My change only works when logging in but not when you're flying around, so I presume something's not right.
Do you know if there's a way for it to pickup the planet name from the line the signals are displayed on please?

{SetState('eddi_context_body_name', event.body)}
{SetState('eddi_context_body_system', event.system)}

{set surfacesignals to []}
{for signal in event.surfacesignals: 
    {set surfacesignals_desc to: 
        {signal.amount} {signal.source}{if signal.amount > 1:s}
    }
    {set surfacesignals to cat(surfacesignals, [surfacesignals_desc])}
}
            
	{OneOf(
        "Picking up {List(surfacesignals)} for {body.bodytype} {P(body.shortname)}",
        "Picking up {List(surfacesignals)} on sensors for {body.bodytype} {P(body.shortname)}",
        "Picking up {List(surfacesignals)} on scanners for {body.bodytype} {P(body.shortname)}",
        "Detecting {List(surfacesignals)} for {body.bodytype} {P(body.shortname)}"
	"Detecting {List(surfacesignals)} on sensors for {body.bodytype} {P(body.shortname)}",
        "Detecting {List(surfacesignals)} on scanners for {body.bodytype} {P(body.shortname)}",
        "{List(surfacesignals)} on sensors for {body.bodytype} {P(body.shortname)}",
        "{List(surfacesignals)} on scanners for {body.bodytype} {P(body.shortname)}",
        "{List(surfacesignals)} detected for {body.bodytype} {P(body.shortname)}", 
        "Sensors detecting {List(surfacesignals)} for {body.bodytype} {P(body.shortname)}",
        "Scanners detecting {List(surfacesignals)} for {body.bodytype} {P(body.shortname)}"
	)}.

Here's an example of the start of the line from the journal file, where I'm trying to pull the Bodyname tags output:
"event":"FSSBodySignals", "BodyName":"DK Leonis AB 3 a"

Thank you

[Tkael]

You can use BodyDetails() to obtain full information about the body given its name. Perhaps something like this?

{set surfacesignals to []}
{for signal in event.surfacesignals: 
    {set surfacesignals_desc to: 
        {signal.amount} {signal.source}{if signal.amount > 1:s}
    }
    {set surfacesignals to cat(surfacesignals, [surfacesignals_desc])}
}

{set body to BodyDetails(event.bodyname, system.systemname)}
{if body:
    {OneOf(
        "{OneOf('Sensors', 'Scanners', '')} {OneOf('Picking up', 'Detecting')} {List(surfacesignals)} on {body.bodytype} {P(body.shortname)}",
        "{List(surfacesignals)} detected on {body.bodytype} {P(body.shortname)}", 
    )}.
|else:
    {List(surfacesignals)} detected.
}

[CMDR-Evolution]

Thank you Tkael!

I haven't been able to test it yet but have copied the functional part from your script, then adjusted the output parts of mine so it's now like this:

{set surfacesignals to []}
{for signal in event.surfacesignals: 
    {set surfacesignals_desc to: 
        {signal.amount} {signal.source}{if signal.amount > 1:s}
    }
    {set surfacesignals to cat(surfacesignals, [surfacesignals_desc])}
}

{set body to BodyDetails(event.bodyname, system.systemname)}
{if body:
	{OneOf(
        "Picking up {List(surfacesignals)} for {body.bodytype} {P(body.shortname)}",
        "Picking up {List(surfacesignals)} on sensors for {body.bodytype} {P(body.shortname)}",
        "Picking up {List(surfacesignals)} on scanners for {body.bodytype} {P(body.shortname)}",
        "Detecting {List(surfacesignals)} for {body.bodytype} {P(body.shortname)}",
	"Detecting {List(surfacesignals)} on sensors for {body.bodytype} {P(body.shortname)}",
        "Detecting {List(surfacesignals)} on scanners for {body.bodytype} {P(body.shortname)}",
        "{List(surfacesignals)} on sensors for {body.bodytype} {P(body.shortname)}",
        "{List(surfacesignals)} on scanners for {body.bodytype} {P(body.shortname)}",
        "{List(surfacesignals)} detected for {body.bodytype} {P(body.shortname)}", 
        "Sensors detecting {List(surfacesignals)} for {body.bodytype} {P(body.shortname)}",
        "Scanners detecting {List(surfacesignals)} for {body.bodytype} {P(body.shortname)}"
	)}.
|else:
	{OneOf(
        "Picking up {List(surfacesignals)}",
        "Picking up {List(surfacesignals)} on sensors",
        "Picking up {List(surfacesignals)} on scanners",
        "Detecting {List(surfacesignals)}",
	"Detecting {List(surfacesignals)} on sensors",
        "Detecting {List(surfacesignals)} on scanners",
        "{List(surfacesignals)} on sensors",
        "{List(surfacesignals)} on scanners",
        "{List(surfacesignals)} detected", 
        "Sensors detecting {List(surfacesignals)}",
        "Scanners detecting {List(surfacesignals)}"
	)}.
}       

I'll be testing it out over the next few days and all being well, I'll then mark your reply as the answer ;).

Fly safe, Evo

[CMDR-Evolution]

Hello again.

Sorry, I've been really busy over the last couple of weeks. Just for a closing comment. I haven't found any issues with this. It works perfectly. Thank you