[Mapbender-dev] mark deprecated functions

[Christoph Baudson]

Hi Thomas,

Thomas Baschetti schrieb:
> Hi,
>
> as more and more functions or modules in mapbender 
> are deprecated i think it would be useful to mark 
> these clearly and visible to all admins. What i'm thinking
> about is something like throwing a notice in the deprecated
> module, e.g. in class_administration.php
> something like this
>
> /**
>  * @deprecated
>  */
> function getGuisByPermission($mb_user_id,$ignorepublic){
>     $e = new mb_notice("administration->getGuisByPermission is deprecated, use user->getGuisByPermission instead!");
>     $user = new User($mb_user_id);
>     return $user->getApplicationsByPermission($ignorepublic);
> }
>
>
> so if the dev or admin reads the logfiles this will get obvious
> (reading logs anyone? ;-)
>
> What do you think about this?
>   

As a developer, I'm indifferent about it (I'm not a user). But I see 
your point.

There are a lot of files to check, and I wouldn't want to go through all 
of them. But if you work on a file that contains a deprecated function, 
just go ahead and add the notice to it.

Maybe we should move the discussion to the user list?

Thanks

Christoph

>
> Ciao
> Thomas Baschetti
>   


-- 
_______________________________________

W h e r e G r o u p GmbH & Co. KG

Siemensstraße 8
53121 Bonn
Germany

Christoph Baudson
Anwendungsentwickler

Fon: +49 (0)228 / 90 90 38 - 15
Fax: +49 (0)228 / 90 90 38 - 11
christoph.baudson at wheregroup.com
www.wheregroup.com
Amtsgericht Bonn, HRA 6788
_______________________________________

Komplementärin:
WhereGroup Verwaltungs GmbH
vertreten durch:
Arnulf Christl, Olaf Knopp, Peter Stamm
_______________________________________