Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Untitled #1

Closed
wants to merge 13 commits into from
Closed

Untitled #1

wants to merge 13 commits into from

Conversation

driusan
Copy link
Collaborator

@driusan driusan commented Dec 1, 2010

Added an extra helper function to utility class which I think is useful for other projects.

Jkat added a commit that referenced this pull request Dec 16, 2013
@Jkat
Merge pull request #1 from Jkat/adding_RBANS_javascript
69fe7d4 
Adding rbans javascript
Jkat referenced this pull request in Jkat/Loris Oct 21, 2014
@samirdas
Merge pull request #1 from driusan/FixSymLinks
920bde2 
Fixed symlinks in MRI scripts
@driusan driusan added the Cleanup PR or issue introducing/requiring at least one clean-up operation label Dec 16, 2014
samirdas pushed a commit that referenced this pull request Sep 25, 2015
@brubo1
Merge pull request #1 from jstirling91/bruno_project
7c00b2d 
make utility function use factory for DB and fix unittest
ted-strauss-K1 pushed a commit that referenced this pull request Feb 5, 2016
@jstirling91
Merge pull request #1 from aces/master
972da4b 
update local master
stellarxo referenced this pull request in stellarxo/Loris May 9, 2016
@driusan
Merge pull request #1 from rathisekaran/EnrollmentReport
d104410 
Code for Enrollment Statistics Report
christinerogers pushed a commit that referenced this pull request Oct 28, 2016
@LeighMac
Merge pull request #1 from aces/17.0-dev
c6b2fe3 
17.0 dev
@gluneau gluneau mentioned this pull request Jan 3, 2017
Merged
@kongtiaowang kongtiaowang mentioned this pull request Mar 27, 2017
Merged
@sruthymathew123 sruthymathew123 mentioned this pull request Mar 27, 2017
Merged
AnyhowStep added a commit to AnyhowStep/Loris that referenced this pull request May 24, 2017
@AnyhowStep
Attempt aces#1 at completing
b4665ec
@xlecours xlecours mentioned this pull request Jun 2, 2017
Merged
ZainVirani referenced this pull request in ZainVirani/Loris Jun 2, 2017
@jacobpenny
Merge pull request #1 from ZainVirani/user_acct_fix
8c5144d 
Fixed user project multi-select display
llevitis pushed a commit to llevitis/Loris that referenced this pull request Jun 8, 2017
xss fix aces#1
482b85a
@MounaSafiHarab MounaSafiHarab mentioned this pull request Jun 19, 2017
Merged
llevitis pushed a commit to llevitis/Loris that referenced this pull request Jul 10, 2017
@leothomas
Merge pull request aces#1 from llevitis/quality_control
62ece69 
[modules] added module name to the 'Clinical' menu (source 0000-00-02-Menus.sql)
@sruthymathew123 sruthymathew123 mentioned this pull request Aug 11, 2017
Merged
@kongtiaowang kongtiaowang mentioned this pull request May 22, 2019
Closed
@kongtiaowang kongtiaowang mentioned this pull request Jul 29, 2019
Closed
@ridz1208 ridz1208 mentioned this pull request Oct 9, 2019
Closed
4 tasks
@kongtiaowang kongtiaowang mentioned this pull request Jan 15, 2020
Merged
ycAbout pushed a commit to ycAbout/Loris that referenced this pull request Jan 31, 2020
@Jkat
Merge pull request aces#1 from cmadjar/excelDumpModification
11c2911 
ExcelDump.php modification on the MRI side
kchatpar pushed a commit to kchatpar/Loris that referenced this pull request Feb 17, 2020
@HenriRabalais
Merge pull request aces#1 from ridz1208/add_000_files
92dae04 
Add 0000* sql
@laemtl laemtl mentioned this pull request Jun 29, 2020
Closed
@laemtl laemtl mentioned this pull request Aug 17, 2020
Closed
@maltheism maltheism mentioned this pull request Sep 4, 2020
Closed
@christinerogers christinerogers mentioned this pull request Oct 27, 2020
Closed
@laemtl laemtl mentioned this pull request Dec 15, 2020
Merged
@maltheism maltheism mentioned this pull request Dec 16, 2020
Merged
@kongtiaowang kongtiaowang mentioned this pull request Feb 17, 2021
Closed
@kongtiaowang kongtiaowang mentioned this pull request Apr 12, 2021
Merged
@ridz1208 ridz1208 mentioned this pull request Jun 16, 2021
Closed
This was referenced Oct 4, 2021
Closed
Merged
driusan pushed a commit that referenced this pull request Oct 14, 2021
@kongtiaowang
[tools] CouchDB_Import_Instruments.php Error (#7689)
735e817 
Fix error

```
PHP Fatal error: Uncaught TypeError: NDB_BVL_Instrument::factory(): Argument #1 ($loris) must be of type LORIS\LORISInstance, string given, called in /var/www/Loris/tools/CouchDB_Import_Instruments.php on line 196 and defined in /var/www/Loris/php/libraries/NDB_BVL_Instrument.class.inc:225
```

In CouchDB_Import_Instruments.php

Fixes #7680
driusan added a commit that referenced this pull request Oct 29, 2021
@driusan
[Instrument] Fix type error in some instruments (#7725)
dc3c184 
When accessing an instrument from RB, I got the error:

```
Fatal error: Uncaught TypeError: Argument 1 passed to ValidatableIdentifier::__construct() must be of the type string, null given, called in
/home/driusan/Code/Loris/php/libraries/NDB_BVL_Instrument.class.inc on line 2863 and defined in /home/driusan/Code/Loris/php/libraries/Valida
tableIdentifier.php:57 Stack trace: #0 /home/driusan/Code/Loris/php/libraries/NDB_BVL_Instrument.class.inc(2863): ValidatableIdentifier->__co
nstruct(NULL) #1 /home/driusan/Code/Loris/src/Middleware/UserPageDecorationMiddleware.php(241): NDB_BVL_Instrument->handle(Object(Laminas\Dia
ctoros\ServerRequest)) #2 /home/driusan/Code/Loris/src/Middleware/PageDecorationMiddleware.php(57): LORIS\Middleware\UserPageDecorationMiddle
ware->process(Object(Laminas\Diactoros\ServerRequest), Object(NDB_BVL_Instrument_aosi)) #3 /home/driusan/Code/Loris/php/libraries/NDB_Page.cl
ass.inc(735): LORIS\Middleware\PageDecorationMiddleware->process(Object(Laminas\Diactoros\ServerRequest), Object(NDB_BVL_Instrument_aosi)) #4
 /home/driusan/Code/Loris/php/libraries/NDB_BVL_In in /home/driusan/Code/Loris/php/libraries/ValidatableIdentifier.php on line 57
```

This fixes the type error so that the instrument page loads.
driusan pushed a commit that referenced this pull request Dec 1, 2022
@ridz1208
[behavioural_qc] Fix fatal error in PHP 8.1 (#8245)
6288e0c 
This fixes the error:

```
<b>Fatal error</b>:  Uncaught TypeError: ValidatableIdentifier::__construct(): Argument #1 ($value) must be of type string, int given, called in /var/www/loris/modules/behavioural_qc/php/models/incompletedto.class.inc on line 126
```

when the module is run on PHP 8.1.
CamilleBeau pushed a commit to CamilleBeau/Loris that referenced this pull request Dec 5, 2022
@jesscall
Merge pull request aces#1 from zaliqarosli/2021_07_27_diagnosis_evolu…
8645ba0 
…tion

update 2021 07 27 diagnosis evolution
@SantiagoTG SantiagoTG mentioned this pull request Apr 11, 2023
Closed
@nicolasbrossard nicolasbrossard mentioned this pull request Apr 13, 2023
Closed
driusan added a commit that referenced this pull request Nov 24, 2023
@driusan
[publication] Fix publication loading (#8975)
7abaa23 
This fixes 2 problems with loading the publication module on Raisinbread:

1.    When trying to load the page, I get the error:

<b>Deprecated</b>:  htmlspecialchars_decode(): Passing null to parameter #1 ($string) of type string is deprecated in <b>/home/driusan/Code/Loris/modules/publication/php/publication.class.inc</b> on line <b>304</b><br />

with PHP 8.2.

2.    The Form.js imports were missing.
driusan added a commit that referenced this pull request Dec 12, 2023
@driusan
[instrument_list] Fix deprecation warning (#8867)
abc7e10 
Fix the warnings:

```
<b>Deprecated</b>:  strtolower(): Passing null to parameter #1 ($string) of type string is deprecated in <b>modules/instrument_list/php/instrument_list_controlpanel.class.inc</b> on line <b>475</b><br />
<br />
<b>Deprecated</b>:  strtolower(): Passing null to parameter #1 ($string) of type string is deprecated in <b>modules/instrument_list/php/instrument_list_controlpanel.class.inc</b> on line <b>514</b><br />
```

in PHP 8.2 and above.
@kongtiaowang kongtiaowang mentioned this pull request Dec 14, 2023
Open
1 task
@charliehenrib charliehenrib mentioned this pull request Feb 29, 2024
Open
@kongtiaowang kongtiaowang mentioned this pull request Mar 27, 2024
Merged
driusan pushed a commit that referenced this pull request Apr 4, 2024
@kongtiaowang
[statistics] bug fix - projectID issue (#9170)
053dff6 
Fixes error on statistics page:

```
[CRITICAL] /var/www/Loris/php/libraries/User.class.inc:476: Uncaught TypeError: User::hasProject(): Argument #1 ($projectID) must be of type ProjectID, string given, called in /var/www/Loris/modules/statistics/php/statistics_site.class.inc on line 134 and defined in /var/www/Loris/php/libraries/User.class.inc:476
```

Fixes #9137
@kongtiaowang kongtiaowang mentioned this pull request Apr 24, 2024
Open
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Cleanup PR or issue introducing/requiring at least one clean-up operation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@driusan