Home pcDTR Forum
If you like the work i do, you know what to do.
Welcome, Guest
Please Login or Register.    Lost Password?

V3.1.4
(1 viewing) (1) Guest
Go to bottomPage: 1
TOPIC: V3.1.4
#204
V3.1.4 5 Months, 3 Weeks ago Karma: 0
Hi again

Followed download link which led to 3.1.4.
Using Joomla 1.5.15, PHP 5.2.6 & 5.2.13, MySQL 5.0.4 & 5.0.89

Problems:

1) On initial installation and whenever cache is cleaned Joomla! dies with a white screen:

Notice: Undefined offset: 1 in /somepath/plugins/system/pcdtr.php on line 156
Fatal error: Class 'JFile' not found in /somepath/plugins/system/pcdtr.php on line 65

This is also the case with 3.1.1 & 3.1.2

2) When the page in refreshed in the above situation the fatal error is resolved but we now have some undefined and array->string errors that have come back:

Notice: Undefined offset: 1 in /somepath/plugins/system/pcdtr.php on line 156
Notice: Array to string conversion in /somepath/plugins/system/pcdtr.php on line 724


Suggestions:

1) Is it necessary to include the styles.css file? This is only being used to create the image isn't it? It was very confusing because my system had the font installed so the text was displaying correctly despite the plugin not processing due to suggestion 2 below... I commented out line 56 to stop it.

2) Can the style.css have normal css definitions where multiple selectors/classes/ids are defined for one set of styles? This caught me out because I had used .contentheading,.componentheading {.... but this didn't parse properly.

3) Would it be worth writing an upgrade/installation routine that would preserve old css file? It is a pain when upgrading... I could help...
mattaulds
Fresh Boarder
Posts: 5
graphgraph
User Offline Click here to see the profile of this user
The administrator has disabled public write access.
 
#205
Re: V3.1.4 5 Months, 3 Weeks ago Karma: 2
Will check that JFile out. I haven't had that problem on my test site yet.

Suggestions:
1) The style.css is needed when you have specified width:, text-align:, background-color: and transparent in the css.

2) Yeah will fix that. Didn't think about that.

3) Sure make a patch and i will add it. I was thinking about that before to. But haven't had time jet to fix it. If you have a github account you can just fork it and make the update and then i will merge it to the master.

And a thx for finding bugs and helping me out to make this script better
flowman
Admin
Posts: 110
graph
User Offline Click here to see the profile of this user
Last Edit: 2010/03/16 11:52 By flowman.
The administrator has disabled public write access.
 
#206
Re: V3.1.4 5 Months, 3 Weeks ago Karma: 2
I uploaded a new 3.1.5 that addressed the issues you had. Want to try it out for me?
flowman
Admin
Posts: 110
graph
User Offline Click here to see the profile of this user
The administrator has disabled public write access.
 
#207
Re: V3.1.4 5 Months, 3 Weeks ago Karma: 2
i found the error with suggestion 2. Its already working. The css parses splits up the css right. But the csstags value from it doesn't. So easy fix it not to use that one

And now it is fixed. So now you can style with .contentheading,.componentheading
flowman
Admin
Posts: 110
graph
User Offline Click here to see the profile of this user
Last Edit: 2010/03/17 00:06 By flowman.
The administrator has disabled public write access.
 
#208
Re: V3.1.4 5 Months, 3 Weeks ago Karma: 0
Good work. Everything is looking good except....

Problem 1) JFile class not found fatal error is still occurring on line 65.

I'm sorry but I've just not had time to dig around your code to look at where it's coming from.
mattaulds
Fresh Boarder
Posts: 5
graphgraph
User Offline Click here to see the profile of this user
The administrator has disabled public write access.
 
Go to topPage: 1
Banner
Copyright © 2010 pcDTR. All Rights Reserved.
Joomla! is Free Software released under the GNU/GPL License.