Do not declare functions in config.php

adam.

On 23/02/2016 21:02:04, Nagy Attila <tylla_at_memetic.org@tylla.hu> wrote:

Not at all. If that were the case than the error would persist even
after I used require_once() for the same unchanged file.
And I can reproduce the error if I simply change the require_once()
function call to a simple require().

Btw. the declaration is there only once. It starts on line 87 and the
ending bracket of the function is on line 102.

So the question remains,
shouldn't require_once/include_once be used in all the places where
referencing config.php?

On 2016-02-23 19:34, Joseph Zancocchio wrote:
> It seems your config.php file was somehow modified during the upgrade
> procedure, such that the custom function you defined in there is being
> redeclared 15 lines later.
>
> It may be as simple as removing the second declaration of
> custom_rewrite_location() starting on line 102, though I suspect you
> may have a few other issues in the config file.
>
> If you have a backup of the config.php file from before the upgrade,
> I'd first try replacing the current one with that backup. Failing
> that, you will need to manually go through the config file to fix any
> other issues introduced during the upgrade. If you can't figure out
> what else, if anything, is messed up, you can share your config.php
> here, with any sensitive info edited out...
>
> On 02/23/2016 01:17 PM, Nagy Attila wrote:
>> Hi all,
>>
>> We are using a customized location rewrite function to split the SNMP
>> location of devices into a human readable and a machine
>> understandable format.
>> We are using the community edition, and on updating to the recent
>> version we were faced with thIS error message:
>>
>> PHP Fatal error: Cannot redeclare custom_rewrite_location()
>> (previously declared in /opt/observium/config.php:87) in
>> /opt/observium/config.php on line 102
>>
>> This is a common error when a php file is included/required more than
>> once, so I made a workaround by renaming config.php to
>> config.user.php and using require_once() in the original config.php
>> to load these directives (it needs absolute path otherways it cannot
>> find the file).
>>
>> Shouldn't require_once/include_once be used in all the places where
>> referencing config.php?
>>
>> Br,
>> Tylla,
>>
>> _______________________________________________
>> observium mailing list
>> observium@observium.org
>> http://postman.memetic.org/cgi-bin/mailman/listinfo/observium
>
> _______________________________________________
> observium mailing list
> observium@observium.org
> http://postman.memetic.org/cgi-bin/mailman/listinfo/observium

_______________________________________________
observium mailing list
observium@observium.org
http://postman.memetic.org/cgi-bin/mailman/listinfo/observium

_______________________________________________
observium mailing list
observium@observium.org
http://postman.memetic.org/cgi-bin/mailman/listinfo/observium