Extension talk:ExtensionClass

Error on 637 and 638

Note, just installed this to be used with Extension:SidebarEx and it comes up with an undefined offset error on lines 637 and 638. I have simply replaced the specifications on those lines with some text. --Dr DBW 01:59, 30 August 2007 (UTC)

I have adjusted the code to include a detection for the condition that causes these warning (now available in SVN Trunk ); the root cause stems from the fact that the repository where one fetches the source code files does not replace the SVN magic words e.g. $Id$.

Jean-Lou Dupont 12:02, 30 August 2007 (UTC)

---

I am trying to install "Extension:SysopSidebar" and it requires "ExtensionClass 'extension'" as a dependancy ...

When i click on "SVN" to get it, i find a bunch of files and I can't see where "ExtensionClass 'extension'" is to be found ??

Can someone help me please ...........

--Dick 15:53, 15 October 2007 (UTC)

Please use Extension:SidebarEx instead now. I will not be supporting Extension:Class nor SysopSidebar anymore. Jean-Lou Dupont 16:07, 15 October 2007 (UTC)

doesn't Extension:SidebarEx need Extension:Class anymore? in the dependencies of sidebarEx it's written that the class is needed, but in the installation instructions only the stubManager looks required. in my wiki the sidebarEx is not working, I don't know why.

--DonGiulio 10:58, 23 February 2008 (UTC)

The documentation is probably out-of-date. Extension:SidebarEx does not require Extension:ExtensionClass. Jean-Lou Dupont 01:46, 24 February 2008 (UTC)

Extension:SidebarEx?

If you cannot DL this on SVN and Extension:SidebarEx should be used then change your page pls. --Subfader 20:42, 17 March 2008 (UTC)

License

The license for this extension is not specified. This is somewhat important since those who base there extensions on this one, are based on the license of this extension. A explicit license such as GPL or Public Domain would be appreciated. Also, useful extension as well as example, Thanks. Servel333 14:55, 4 August 2009 (UTC)

Bug

line 260 should be like

		// v1.5 feature
		foreach (self::$hookList as $index => $hookName)
		{
			if (!empty($replaceHookList))
			{
				$replaceFlag = in_array( $hookName, $replaceHookList);
			}
			else
			{
				$replaceFlag = false;
			}
				
					
			if ( in_array( 'h'.$hookName, get_class_methods($this->className) ) )
			{
				if ( $replaceFlag )
					$wgHooks[$hookName][count($wgHooks[$hookName])-1] = array( &$this, 'h'.$hookName );
				else
					$wgHooks[$hookName][] = array( &$this, 'h'.$hookName );
			}

or maybe

		// v1.5 feature
		foreach (self::$hookList as $index => $hookName)
		{
			if (!empty($replaceHookList))
			{
				$replaceFlag = in_array( $hookName, $replaceHookList);
                if ( in_array( 'h'.$hookName, get_class_methods($this->className) ) )
                {
                    if ( $replaceFlag )
                        $wgHooks[$hookName][count($wgHooks[$hookName])-1] = array( &$this, 'h'.$hookName );
                    else
                        $wgHooks[$hookName][] = array( &$this, 'h'.$hookName );
                }
			}
		}

to prevent Error with $replaceFlag