Yep - it's the older pull... the one that inspired me to find the better
(not yet merged) approach.  So we're on the same page :-).

Sent from my Swiss Army Phone
On Apr 21, 2013 12:27 AM, "Bharat Mediratta" <notifications@github.com>
wrote:

> In modules/gallery/classes/Gallery/Model/Item.php:
>
> >    public $data_file = null;
> >    private $data_file_error = null;
> >
> >    public function __construct($id=null) {
> > +    if (Module::is_active("tag")) {
>
> I'm not sure if I'm processing these pull requests out of order - but from
> an architectural point of view we really want to avoid this type of
> construct. The main gallery module should not know about other modules. I
> think the other change where we do event based relationship additions is
> the right way to go here.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/gallery/gallery3/pull/295/files#r3886038>
> .
>