Skip to content

Commit

Permalink
Fixed error caused by ProgressBar's old constructor name.
Browse files Browse the repository at this point in the history
- Full error message: "Deprecated: Methods with the same name as their class will not be constructors in a future version of PHP; ProgressBar has a deprecated constructor in /www/git/LOVD3/src/class/progress_bar.php on line 41".
  • Loading branch information
ifokkema committed Jan 27, 2017
1 parent 8e23fb1 commit fe06dbb
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions src/class/progress_bar.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@
* LEIDEN OPEN VARIATION DATABASE (LOVD)
*
* Created : 2010-09-10
* Modified : 2016-10-14
* For LOVD : 3.0-18
* Modified : 2017-01-27
* For LOVD : 3.0-19
*
* Copyright : 2004-2016 Leiden University Medical Center; http://www.LUMC.nl/
* Programmers : Ing. Ivo F.A.C. Fokkema <I.F.A.C.Fokkema@LUMC.nl>
* Ing. Ivar C. Lugtenburg <I.C.Lugtenburg@LUMC.nl>
* Copyright : 2004-2017 Leiden University Medical Center; http://www.LUMC.nl/
* Programmers : Ivo F.A.C. Fokkema <I.F.A.C.Fokkema@LUMC.nl>
* Ivar C. Lugtenburg <I.C.Lugtenburg@LUMC.nl>
*
*
* This file is part of LOVD.
Expand Down Expand Up @@ -46,7 +46,7 @@ class ProgressBar {



function ProgressBar ($sID = '', $sMessageInit = '', $sMessageDone = '')
function __construct ($sID = '', $sMessageInit = '', $sMessageDone = '')
{
// Default constructor.

Expand Down

2 comments on commit fe06dbb

@mkroon1
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this fix #156 ? Or are there still possible other classes using old-style constructors?

@ifokkema
Copy link
Member Author

@ifokkema ifokkema commented on fe06dbb Jan 27, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah haha yes, it does, thanks :) Totally overlooked this issue!
I checked, there are no other old-style constructors.

Please sign in to comment.