Sanitize location admin action name to ignore format strings

[goyal1092]

Description

In location admin we are using category name in action description It threw error if there was a % sing in name as python treated it as format string. Fixed by adding double % sign if % is present. https://stackoverflow.com/questions/26393895/python-how-to-escape-s/26393941#26393941 I think it's most feasible solution as we do not want to remove % sign

Related Issue

https://wazimap.atlassian.net/browse/WNCM-447

How to test it locally

• Add % sign in any collection name.
• Try to open location admin list page
• Page should open up without any problems

Changelog

Added

Updated

Removed

Checklist

• [x] 🚀 is the code ready to be merged and go live?
• [x] 🛠 does it work (build) locally

Pull Request

• [x] 📰 good title
• [x] 📝good description
• [x] 🔖 issue linked
• [x] 📖 changelog filled out

Commits

• [x] commits are clean
• [x] commit messages are clean

Code Quality

• [x] 🚧 no commented out code
• [x] 🖨 no unnecessary logging
• [x] 🎱 no magic numbers
• [x] black was run locally (as part of the pre-commit hook)

Testing

• [x] ✅ added (appropriate) unit tests
• [x] 💢 edge cases in tests were considered
• [x] ✅ ran tests locally & are passing

[jbothma]

Please add a test that verifies that a category with % in its name does indeed result in being rendered as % on the location list view to detect if/when they change this.

As far as I can tell, replacing % with %% should make this safe.

[goyal1092]

Fixed failing test and added comment according jd's review @michaelglenister