https://bugzilla.wikimedia.org/show_bug.cgi?id=26857
T. Gries changed:
What|Removed |Added
See Also||https://bugzilla.wikimedia.
https://bugzilla.wikimedia.org/show_bug.cgi?id=26857
Chad H. changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
https://bugzilla.wikimedia.org/show_bug.cgi?id=26857
--- Comment #8 from Roan Kattouw 2011-01-24 01:52:17
UTC ---
(In reply to comment #7)
> I am having the same issue after trying to install the Collection extension
> from svn trunk on a 1.16.1 upgraded install.
>
> Notice: Undefined variable:
https://bugzilla.wikimedia.org/show_bug.cgi?id=26857
Alain Picard changed:
What|Removed |Added
CC||pic...@castortech.com
--- Comment #7 fr
https://bugzilla.wikimedia.org/show_bug.cgi?id=26857
--- Comment #6 from Brion Vibber 2011-01-22 03:55:32 UTC ---
Ok, so if the only thing we're going to take out of it is $wgHooks entries for
schema updates, then explicitly global'ing $wgHooks is probably ok for
functional purposes.
We still ha
https://bugzilla.wikimedia.org/show_bug.cgi?id=26857
--- Comment #5 from Chad H. 2011-01-22 02:42:09 UTC
---
(In reply to comment #4)
> If they're creating / adding to a local variable called '$wgHooks' within
> Installer::includeExtensions(), I don't think that'll trigger anything at
> install
https://bugzilla.wikimedia.org/show_bug.cgi?id=26857
--- Comment #4 from Brion Vibber 2011-01-22 02:35:33 UTC ---
If they're creating / adding to a local variable called '$wgHooks' within
Installer::includeExtensions(), I don't think that'll trigger anything at
install time.
They would need acce
https://bugzilla.wikimedia.org/show_bug.cgi?id=26857
--- Comment #3 from Chad H. 2011-01-22 02:28:22 UTC
---
(In reply to comment #2)
> Now, if the point of including the extensions at this time is to get hooks set
> up so it'll do the database installation, I don't think this is going to work
>
https://bugzilla.wikimedia.org/show_bug.cgi?id=26857
--- Comment #2 from Brion Vibber 2011-01-22 02:16:01 UTC ---
The installer require()s extension files from within the
Installer::includeExtensions() method, so I think any reference to global
configuration variables directly in the extension fi
https://bugzilla.wikimedia.org/show_bug.cgi?id=26857
Roan Kattouw changed:
What|Removed |Added
CC||roan.katt...@gmail.com
--- Comment #1 f
10 matches
Mail list logo