Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#15546] - Fix incorrect calendar paths

? Pending

User tests: Successful: Unsuccessful:

avatar saharin88
saharin88
25 Apr 2017

Pull Request for Issue #15441 (comment).

Summary of Changes

Fixes incorrect paths for resources

avatar saharin88 saharin88 - open - 25 Apr 2017
avatar saharin88 saharin88 - change - 25 Apr 2017
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Apr 2017
Category ⇒ Libraries
avatar wilsonge wilsonge - change - 25 Apr 2017
Labels Added: ?
avatar wilsonge wilsonge - change - 25 Apr 2017
Title
…
Update behavior.php
Fix incorrect calendar paths
avatar wilsonge wilsonge - edited - 25 Apr 2017
avatar wilsonge wilsonge - edited - 25 Apr 2017
avatar wilsonge
wilsonge - comment - 25 Apr 2017

I've fixed the CSS path here and added the link to the correct issue :)

avatar dgt41
dgt41 - comment - 25 Apr 2017

I have tested this item ✅ successfully on 318cec9

To test replace in isis/index.php

defined('_JEXEC') or die;

with

defined('_JEXEC') or die;
JHtml::_('behavior.calendar');

and check the page source for the css and the scripts files

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15546.
avatar dgt41 dgt41 - test_item - 25 Apr 2017 - Tested successfully
avatar wilsonge
wilsonge - comment - 25 Apr 2017

@brianteeman are you able to test this please?

avatar brianteeman
brianteeman - comment - 26 Apr 2017

And this is why merging a pr without any tests was a bad idea!!!!

Using the test instructions from @dgt41 this PR works as intended

avatar wilsonge wilsonge - change - 26 Apr 2017
The description was changed
Status Pending ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2017-04-26 08:24:03
Closed_By ⇒ wilsonge
avatar wilsonge wilsonge - close - 26 Apr 2017
avatar wilsonge wilsonge - merge - 26 Apr 2017
avatar gwsdesk
gwsdesk - comment - 26 Apr 2017

I confirm this as well. This was also posted to me by @Markus Bopp of Crosstec just a few ago.This works with the method as described by @dgt41

Add a Comment

Login with GitHub to post a comment