Fix formatting of Upcoming Audit Email Body

[yusuf-basith]

The Asset Notes field included in the Upcoming Audit Email is a multi-line text field. Hence there is a high chance of having a line break in this field.

Currently, the asset notes are passed as it is in the blade templates to the markdown format, which leads to broken HTML table in the final email, when there are line breaks.

Vibe coding with GIthub copilot shows that we can use $escaped_notes = nl2br(e($asset->notes)); to convert the new lines in the asset notes to <br> tag, which is one of the known ways to manage multi-line cells in markdown tables.

Copilot also told to use {!! $escaped_notes !!} while inserting it in the table to escape HTML tags.
While trying to understand the code, it told about XSS vulnerabilities. So I let the professionals deal with the right way of working.

[uberbrady]

I'd tend towards something like:

@foreach(explode($lines,"\n") as $line) 
{{ $line }}<br><br>
@endforeach

[uberbrady]

If you put carriage-returns in the asset notes field, you should be able to replicate this output. Make sure you can make it break first, then make sure you can do the fix.

[yusuf-basith]

Sample to replicate the issue:

{
    "id": 4209,
    "name": "My Lapotop",
    "asset_tag": "0001234567",
    "serial": "ABCDEFG",
    "notes": "Dell Order No: 123456\r\nP/O No: DELL/123/01/2024\r\n\r\nExpected Delivery Date: 29/02/2024",
    "available_actions": {
        "checkout": true,
        "checkin": true,
        "clone": true,
        "restore": false,
        "update": true,
        "audit": true,
        "delete": false
    }
}

[yusuf-basith]

Sorry again!