Les données pour l'installation ne semblent pas à jour (deux champs de la table folders sont évoqués qui ne figurent pas dans la table) et le fichier global openillink.sql est manquant

Hello,

I have been trying to set up OpenILLink with MySQL 5.7 and I was having issues creating orders because of inserting blank strings for 'illinkid' and when various date fields are not filled in. To fix this I removed the MySQL modes STRICT_TRANS_TABLES, NO_ZERO_IN_DATE and NO_ZERO_DATE from the default MySQL modes in 5.7 and I was getting no issues.

Programatically it can be fixed by removing 'illinkid' from the 'orders_new' INSERT query and letting auto-increment handle it, and setting empty dates to Null instead of '0000-00-00'. Would there be a reason for not doing this?

Add support to send multiple orders from an uploaded list of references (provided in format such as RIS or EndNote XML, etc.).

Libraries such as https://github.com/hash-bang/RefLib could be used to parse input files.

Provide possibility to edit the list of uploaded references before they are sent.

Optionally, find a way to tag these orders specifically such that they can be processed independently, if needed.

It would be nice to add an alert when the paper is available in open access, using the oaDOI API (https://oadoi.org/api), for the users at the moment they submit the order form or as an alert for managers when they open the details page.

Due to the ordering of the require{_once} instructions, it can happen that debugOn variable is not correctly initialized when used. For eg:

Notice: Undefined variable: debugOn in includes\orders_list.php on line 40

When submitting new orders, duplicates can be detected using combined year+volume+page. Pages are sometimes mentionned using different type of ranges on the source. For eg:

• 217-20
• 217-220

It has been suggested to consider only the starting page when searching for matching orders in the DB.

When uploading a file with a list of references to order, try to parse and import the URL data if it exists in the provided file (EndNote, RIS, etc.). It can be added to the note field.

For eg. already parsed from Endnote XML, but not stored.

In order to test OpenILLink on a wide range of browsers using BrowserStack and its free licence for open source projects, add a logo on the README.md file.

The new fields added to the links table are missing in the line :

https://github.com/openillink-project/openillink/blob/master/openillink/data/openillink_data_links.sql#L7

I prepare a new pull request to test the work with branches and collaboration from the UNIGE fork

After commit 6c4f236 duplicate orders are highlighted for the admins. Since a duplicate warning is currently only stored as text, the highlighting works based on the existence of the sentence "Warning. Possible duplicate of the order" (which is stored in the language chosen by the customer, for eg. "English"). When the OpenILLink interface is displayed in a different language (for eg. "French") by the "admin", the highlighting searches for the text "Attention. Possible doublon de la commande." which won't match the English string.

Add a configuration variable in order to enable/disable administration of "folders" via the web interface.

It is proposed that administration of "folders" via the web is not (completely) enabled until a config variable (eg. configEnableFoldersEdit ) is set to true. When it is not enabled, the textarea for the request would be marked as "read-only", and the business logic would not update the corresponding column (ditto when creating filters).

Currently when a new configuration variable is added to config.php, every site admin that upgrade will need to specifically merge her config file with the default, upgraded config.php, i.e. must add any new configuration variable to ensure that OpenILLink will run.

Make it possible for site admins to configure options in a file that overrides the default OpenILLink configuration, and inherit options that are not specifically configured.

This could be done as simply as having config.php to first import from a default_config.php file where all configurations are defined.

OpenILLink localizes its interface using array variables defined in translations.php. An alternative way to handle internationalization would be to use gettext.

It is proposed to switch OpenILLink to gettext in order to:

• ease maintenance of translatable strings
• make it possible for non-technical users to provide translations
• provide a clearer separation between business logic and user interface
• adopt a widely used practice to handle I18N
• etc.

One drawback of using gettext is that it requires to have the gettext module enabled on the server running OpenILLink. Though it seems wideley installed by default, it is interesting to check if it is the case for production sites currently (or planning to) running OpenILLink. This can be checked by inspecting the result of phpinfo().

Find duplicate orders based on DOI, in addition to PMID and year+vol+page.

Les codes d'accès pour Basel et la NLM doivent pouvoir être personnalisées par chaque bibliothèque sinon ce sont les codes de la bibliothèque centrale qui sont envoyés!

A l'origine je recuperais les données de l'URL et non pas de la config : exemple pour basel
nouveau
avant