Please post any bug reports for 9.14.xx here
Announcement
Collapse
No announcement yet.
Miva Merchant 9.14.x Bug Reports
Collapse
This is a sticky topic.
X
X
-
Not sure if this is specific to 9.14 or has been present since 9.13, but:
For JavasScript/CSS Resources, a feature was added in 9.13 that appended an updated timestamp parameter T=XXX when the resource was edited in the admin dashboard. This allowed the use of the dashboard to edit the file and prevented caching issues when using the <mvt:item> method of including the file.
What I'm noticing is that if I edit and save a file that has a local path of say, /js/filename.js, this timestamp parameter is not updated.
However, if I edit and save the same file that has a local path of js/filename.js (and thus resides in the mm5 folder), the timestamp parameter updates correctly for both the file I just edited and the file located in /js/.
Not sure if you are intending all local resource files to reside in mm5 or if this is an oversight, but would be good information either way.
Related to bug fix 26774 I believe.
-
new_user2018 There's definitely a bug there (more on this in a second), but looking at the original commit and what the code is actually doing, the intention was to have the files exist in the base path (usually /mm5/ unless it was changed). The reason being that different configurations could lead to unreadable file paths when attempting to calculate the last modified time.
The bug is that the way we build the file path based on the JavaScript resource path means that absolute paths are still getting the base path prepended, which is incorrect (ie, js/filename.js is converted to /mm5/js/filename.js when reading the relative filepath on the server, and /js/filename.js is converted to /mm5/js/filename.js, rather than just looking at /js/filename.js). That's what is causing your timestamp parameter to suddenly update when you modify the relative resource. I'll file a bug to fix the incorrect prepend. Whether or not we'll alter it to support reading the absolute file paths, I'm not sure. In the mean time, using relative paths, which will prepend the base href path, will work and give you the correct timestamp cache-busting parameter.
- 1 like
Comment
-
Originally posted by kayakbabe View PostMarketing Feeds output file naming forces .txt . I set the output file name to be myexample.csv and the output is always being named myexample.csv.txtDavid Carver
Miva, Inc. | Software Developer
Comment
-
NEW ANNOYING BUG:
User Interface > Search Settings > Menu Item Template > Advanced
I can not "select >" custom fields. I click one and it visually moves over to the right but when I click UPDATE it's not there anymore.
We confirmed this on multiple PCs and multiple sites.Colin Puttick
Miva Web Developer @ Glendale Designs
Comment
-
Originally posted by dcarver View PostWhat module is this? Only feed modules we have are Sitemap and Template Feed. Template Feed can output to a customizable file and from my testing it names the file correctly.
I added the following to the top of template before calling the template feed item:
Code:<mvt:assign name="g.return_value" value="miva_output_header('Content-type','text/csv; charset=utf-8')" /> <mvt:assign name="g.return_value" value="miva_output_header('Content-Disposition', 'attachment; filename=output.csv')" /> <mvt:item name="templatefeed" />
maybe it's actually a browser issue?
Comment
-
not sure if this is new, or different. But...
mvga.js script is not always outputting what I think it should for the pageview variable. For example, the invoice page is seen as merchant.mvc not invoice.html. When I think it should be getting invoice.html. invoice.html is what appears in the browser url bar... but it isn't what is populating into the ga( 'send', 'pageview',
So our checkout funnel goal isn't able to complete. Obviously if there are other url problems with the site, we don't want merchant.mvc to be what is tracked.
In the advanced view of the ga_tracking template... I see this..
<mvt:if expr="( g.UI_Exception NE 1 ) OR ( g.Session:cache:last_ui_exception EQ 'order_invoice' )">
ga( 'send', 'pageview', { 'page':'<mvt:item name="ga_jsencode" param="ga_tracking:url_override_unencoded" />' } );
</mvt:if>
What is the purpose of having an exception for the order_invoice?
Last edited by kayakbabe; 10-01-19, 10:50 AM.
- 1 like
Comment
-
Smart breadcrumbs ignore the "Always Link to This Page Using HTTPS" flag on template pages.
I discovered this because I have a page template with a form that I want to set the action to itself using its canonical url.
I looked in the token list to find the correct token. It seemed to be the &mvt:breadcrumbs:current_item:link; would be the one to use.
then I tested and realized the &mvt:breadcrumbs:current_item:link; is outputting http:// instead of https://
I found out, that is the story with ALL my breadcrumbs throughout my site; And, becuase this code
<mvt:item name="customfields" param="Write_Basket('continue_shopping', l.settings:breadcrumbs:current_item:link)" />
in the http profile sets up the button for continue shopping in the basket page, it also has an insecure link.
All this jumping around between http and https is also throwing off my google analytics.
The smart breadcrumbs should respect the "always link to this page using https" flag. But they ignore it completely.
They get their base url from URI Management:Settings:Runtime:URL Prefix, and do not check the template flag.
If they did check the template flag and the flag is checked, then they should get the base url from URI Management:Settings:Runtime:Secure URL Prefix instead. They don't.
these variables are problems, they should been secure links since the "always link to this page using https" is checked onOne solution is to set the url value in the URI Management:Settings:Runtime:URL Prefix field to an https url.l.settings:breadcrumbs:current_item:link &mvt:breadcrumbs:current_item:link; http:// l.settings:breadcrumbs:links[1]:link &mvt:breadcrumbs:links[1]:link; http:// l.settings:cats:link &mvt:cats:link; http:// l.settings:seo_settings:urlprefix &mvt:seo_settings:urlprefix; http://
But that might not be what the store owner really wants to do. So, I think this is a bug.Last edited by kayakbabe; 10-17-19, 12:15 AM.
Comment
-
Originally posted by dreamingdigital View PostNEW ANNOYING BUG:
User Interface > Search Settings > Menu Item Template > Advanced
I can not "select >" custom fields. I click one and it visually moves over to the right but when I click UPDATE it's not there anymore.
We confirmed this on multiple PCs and multiple sites.David Carver
Miva, Inc. | Software Developer
Comment
-
I'm going to put this in the bug post because it's totally in a spot that makes no sense.
The checkbox that triggers the runtime.js to be defer/async --- yeah, that. --- yes there is docs.
https://docs.miva.com/how-to-guides/deferred-javascript
Why? is the checkbox toggle for defer / async of the core miva JavaScript on THAT search settings screen!?? Please, move it to the domain settings > Site config > Javascript section. Ugg... wow.... sorry.
I had to fight mysterious browser issues in Chrome of all things until I realized the version of Shadows I checked out had this checkbox on by default and the scripts that I needed to be timed properly were async and I didn't know cause that checkbox is hidden in the Search settings page.
Rant over. Sorry. This change will be better for everyone else other than me. I already know where this checkbox is hidden.Colin Puttick
Miva Web Developer @ Glendale Designs
- 2 likes
Comment
Comment