Dave MacFarlane

Results 81 issues of Dave MacFarlane

The PaginationLinks was the first React component ever written in LORIS. As such, it is currently a class based component written in Javascript. It also dated from a time where...

Language: PHP
Language: Javascript

The dates on the menu filter in the publication module are not formated according to the user's locale as other places. They should be formatted using an Intl.DateTimeFormat formatter.

Remove the old CouchDB based "dqt" module which has been replaced by the "dataquery" module. All projects that I've asked who still use the "dqt" have overrides, so there is...

Language: SQL
RaisinBread
Language: PHP
Language: Javascript
Module: dqt
Module: dataquery

The "Terms of Use" are not visible on the login page in the mobile view. When requesting an account, it provides an error saying the terms of use were not...

Category: Bug

There are a number of functions in Utility.class.inc that have been there for years. get??Instruments have printed a deprecation warning for 4 years. Other methods have been throwing exceptions. It's...

Language: PHP

This speeds up `make testdata` (and the GitHub Actions tests in general) faster by converting the format of loading from one `INSERT` statement per row to a bulk `LOAD DATA...

Language: SQL
State: Needs work
RaisinBread
Language: PHP

This updates the NDB_Form class to respond with a `303 See Other` after a successful page and redirect to the same page. This (in addition to being a better practice)...

State: Blocked
Language: PHP
Module: statistics
Module: api
Module: candidate_profile
Module: conflict_resolver
Module: instruments

The parameter_type_override from the dictionary and datadict module allows a field description to be overridden. However, there is one override per language. This does not make sense in a multi-lingual...

All logged in user session data is private. There is no reason to restrict the session_cache_limiter to the API. (In fact, it is a bad idea.)

Category: Bug
Category: Security
Language: PHP
Difficulty: Trivial

The showPasswordExpiryScreen method in SinglePointLogin redirects to a page that does not exist, and does it in a very hacky way. This removes it to avoid confusion with the login...

Category: Cleanup
Language: PHP
Difficulty: Medium