Enhancing photo renaming with custom expressions

[VitorVieiraZ]

Setting Photo name format in Photo attachments configuration previously did not accept formats beginning with '/', such as '/photos/'||@mergin_username||'-'||@layer_name||to_string("count")||'_'||format_date("date",'yyyy-MM-dd_Thh.mm.ss'). Now, it is possible to configure expression with leading slashes. Mergin Maps will automatically remove any slashes at the beginning of the photo name format during data processing.

Fixes #3415

[tomasMizera]

Looks good. Can we add a test here please?

[github-actions]

Pull Request Test Coverage Report for Build 9653861643

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 8 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-0.05%) to 59.919%

---

Files with Coverage Reduction	New Missed Lines	%
input/core/merginapi.cpp	8	78.49%

Totals
Change from base Build 9583371861:	-0.05%
Covered Lines:	7675
Relevant Lines:	12809

---

💛 - Coveralls

[github-actions]

Pull Request Test Coverage Report for Build 9653861643

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 8 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-0.05%) to 59.919%

---

Files with Coverage Reduction	New Missed Lines	%
input/core/merginapi.cpp	8	78.49%

Totals
Change from base Build 9583371861:	-0.05%
Covered Lines:	7675
Relevant Lines:	12809

---

💛 - Coveralls

[github-actions]

Pull Request Test Coverage Report for Build 9653861643

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 8 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-0.05%) to 59.919%

---

Files with Coverage Reduction	New Missed Lines	%
input/core/merginapi.cpp	8	78.49%

Totals
Change from base Build 9583371861:	-0.05%
Covered Lines:	7675
Relevant Lines:	12809

---

💛 - Coveralls

[tomasMizera]

Almost there! :)

[github-actions]

Pull Request Test Coverage Report for Build 9678435592

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 9 unchanged lines in 2 files lost coverage.
• Overall coverage decreased (-0.04%) to 59.927%

---

Files with Coverage Reduction	New Missed Lines	%
input/core/merginuserinfo.cpp	1	75.18%
input/core/merginapi.cpp	8	78.49%

Totals
Change from base Build 9583371861:	-0.04%
Covered Lines:	7676
Relevant Lines:	12809

---

💛 - Coveralls

[github-actions]

Pull Request Test Coverage Report for Build 9678435592

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 9 unchanged lines in 2 files lost coverage.
• Overall coverage decreased (-0.04%) to 59.927%

---

Files with Coverage Reduction	New Missed Lines	%
input/core/merginuserinfo.cpp	1	75.18%
input/core/merginapi.cpp	8	78.49%

Totals
Change from base Build 9583371861:	-0.04%
Covered Lines:	7676
Relevant Lines:	12809

---

💛 - Coveralls

[github-actions]

Pull Request Test Coverage Report for Build 9678435592

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 9 unchanged lines in 2 files lost coverage.
• Overall coverage decreased (-0.04%) to 59.927%

---

Files with Coverage Reduction	New Missed Lines	%
input/core/merginuserinfo.cpp	1	75.18%
input/core/merginapi.cpp	8	78.49%

Totals
Change from base Build 9583371861:	-0.04%
Covered Lines:	7676
Relevant Lines:	12809

---

💛 - Coveralls

[tomasMizera]

Thanks for addressing my comments! :)