Sure, that sounds reasonable to me.  Should I change the code?

Bill
On May 12, 2014 7:19 PM, "George Wilson" <notifications@github.com> wrote:

> In administrator/templates/hathor/html/com_templates/templates/default.php:
>
> > @@ -80,6 +80,21 @@
> >  		<tbody>
> >  		<?php foreach ($this->items as $i => $item) : ?>
> >  			<tr class="row<?php echo $i % 2; ?>">
> > +		            <?php
> > +		                    if (!$item->xmldata){
> > +		                        $xmldata = new JObject;
>
> I understand the need for cloning the variable to $xmldata - but could be
> not just initialise the new var as a JRegistry rather than JObject?
>
> I think that would work and still target all your cases successfully
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/joomla/joomla-cms/pull/3589/files#r12559332>
> .
>