Doing complete cache reset calls OnAfterConfigRead twice

[Alexander Obuhovich]

When doing a complete cache reset an admin tree sections are rebuilt as well.

However in the process OnAfterConfigRead events are called too resulting code listed there executed once more.

This doesn't do much harm, unless you're adding values to unit config (e.g. adding LEFT JOIN to sql) instead of replacing them. As a result double LEFT JOIN is added causing SQL error.

--
Best Regards,

http://www.in-portal.com
http://www.alex-time.com

[Dmitry Andrejev]

Easy to fix?

DA

[Alexander Obuhovich]

Doesn't seem like so.

I can't prevent calling of OnAfterConfigRead events, because new sections can be added from there and existing sections can be altered.

And I can't prevent OnAfterConfigRead execution during regular cache rebuild since I don't know if section cache rebuild will be done after that and before final cache save to database.

Since section cache rebuild can be done on it's own it requires complete unit config rescan, but doesn't save rescan result to database.

--
You received this message because you are subscribed to the Google Groups "In-Portal Bugs Team" group.
To view this discussion on the web visit https://groups.google.com/d/msg/in-portal-bugs/-/hPBsIU8C9I4J.
To post to this group, send email to in-port...@googlegroups.com.
To unsubscribe from this group, send email to in-portal-bug...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/in-portal-bugs?hl=en.