samvera / questioning_authority

Question your authorities
Other
54 stars 30 forks source link

add ability to encode the variable value #195

Closed elrayle closed 5 years ago

elrayle commented 5 years ago

Encoding is primarily used to encode URIs that are part of the URL path instead of being a parameter. But the generalized processing allows any variable to be marked as requiring URL encoding.

Also making all methods private unless they are needed in other classes. In this refactor, that effect #default and #required?

The encoding support was in the original term_url_with_replacements which is obsolete with PR #187. That functionality should have been pulled into variable maps then. This PR incorporates this functionality in a broader way that allows any variable to be encoded based on configurations.