p575 Unify description field

[akleshchev]

Landmarks permit newlines in the description, but also permit pipes (which breaks notecards), other items permit neither. Unified requirements.

P.S. I will create a ticket for CopyInventoryFromNotecard separately and a bit later.

[Copilot]

Pull request overview

This PR standardizes the validation rules for inventory/landmark description fields so they can contain newlines while still disallowing the pipe (|) character (which is used as a legacy delimiter and can break notecard-style serialization). It also hardens legacy inventory item stream import/export to better preserve multi-line descriptions by escaping/unescaping newlines.

Changes:

• Introduces/renames a text validator to allow ASCII + newline while excluding |, and wires it up via XUI prevalidator attributes.
• Updates the validator error string for the new “newline + ASCII (no pipe)” rule.
• Updates LLInventoryItem legacy stream import/export to escape/unescape newlines and reassemble multi-line desc fields split across lines.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
indra/newview/skins/default/xui/en/strings.xml	Adds a new validator error string for the no-pipe, newline-allowing validator.
indra/newview/skins/default/xui/en/sidepanel_item_info.xml	Switches the description field to a newline-allowing, no-pipe validator.
indra/newview/skins/default/xui/en/panel_landmark_info.xml	Switches landmark notes to the newline-allowing, no-pipe validator.
indra/newview/skins/default/xui/en/floater_create_landmark.xml	Switches landmark creation notes to the newline-allowing, no-pipe validator.
indra/llui/lltextvalidate.h	Renames the exported validator symbol to the new “newline + no pipe” variant.
indra/llui/lltextvalidate.cpp	Implements the “newline + ASCII excluding pipe” validator and registers it as ascii_with_newline_no_pipe.
indra/llinventory/llinventory.cpp	Escapes newlines on export, reassembles multi-line desc on import, and unescapes back to the original content.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.