justinbeaty / module-gene-braintree

OpenMage compatible fork of Gene_Braintree module
7 stars 5 forks source link

Order Details Page HTML Issue #6

Open johnvoncolln opened 1 year ago

johnvoncolln commented 1 year ago

Hi,

Sorry to put this here, but not sure where else to ask. I've been playing with the Braintree extension on our staging server and any new orders placed using Braintree (sandbox), the order details page is skewed and bunched up to one side. Orders that do not use Braintree, old or new, look normal. For instance, I placed another order with Check payment and it looks fine.

First image is of an order using a different payment method, second image is using braintree.

I'm not familiar enough with Openmage / Magento to do much worthwhile troubleshooting... Any ideas?

Screenshot 2023-08-29 at 11 55 07 AM Screenshot 2023-08-29 at 11 57 26 AM
johnvoncolln commented 1 year ago

I've compared the two html files and see that with the Braintree transaction, it's throwing:

<div class="columns ">
                    <div class="side-col" id="page:left">

above the html doctype declaration tag, and then it's missing further down the page. Note the space next to "columns" as well.

justinbeaty commented 1 year ago

@johnvoncolln That's pretty strange, it's supposed to add a block like this:

image

I'm not sure why it's not, perhaps it could be a module conflict though. Are there any other modules related to payment methods or the order screen you could try to disable (in /etc/modules/*.xml) to see if it is resolved?

johnvoncolln commented 1 year ago

I've gone through all 3rd party extensions and disabled, no change. I had already removed most of the other payment extensions. Any other ideas?

johnvoncolln commented 1 year ago

Line 20, in: /app/design/adminhtml/default/default/template/gene/braintree/creditcard/info.phtml

It's echoing out the values of the payment information, and it's not happy for whatever reason:

<td><?php echo nl2br(implode($this->getValueAsArray($_value, false), "\n"))?></td>

replaced it with:

<td><?php echo $_value ?></td>

Works fine now. Maybe it's something to do with the legacy extension?