John Mitchell Posts:3471


| | 08/01/2008 11:20 AM |
| | Sure, you can send me a private message by clicking on the message icon under my username. | | | |
|
paco
 Posts:27

| | 08/06/2008 7:25 AM |
| Hello John, thanks for making it work.
How about moving the viewstate down. I saw in your source code that the viewstate is down indeed.
In my config file the rule is still commented (off). I tried to turn it on after your changes but this rule is still not working correctly. It generates a white page. Can you take a look at this too?
Regards, Paco | | | |
|
John Mitchell Posts:3471


| | 08/06/2008 7:38 AM |
| Hi Paco, You're welcome.
Ok, I'll try to figure out why the viewstate rule isn't working on your site and let you know. | | | |
|
paco
 Posts:27

| | 08/14/2008 7:37 AM |
| Hello John, Do you have some news? Regards, Paco | | | |
|
John Mitchell Posts:3471


| | 08/14/2008 2:11 PM |
| Hi Paco, Could you save your Page Source and attach it here please. I'm not able to recreate this problem with that replacement rule on any other site, but I'll try running yours through a debugger. | | | |
|
paco
 Posts:27

| | 08/15/2008 12:07 PM |
| Hello John, I updated my website with the latest snapsis pageblaster settings that you made on the test server. After that I had to change one setting. Reason is that I am using the Aggregator module from DnnStuff that makes it possible to work with Tabs within a page. I also use it for localization.
Because of the use of this module in combination with Pageblaster I had to change the rule "Move dnn js links to Head". I had to set the "repeatTimes" to "1". The error I got was a javascript error saying 'DNN' is undefined. Maybe you can take a look at this too? I left the setting on the test server to "99" so you can see the error. To see the error you can go to e.g. the page "website proposal" where the Aggregator module is being used. I pasted the source as you requested, I believe you mean the output that is being sent to the browser. If you want to have some other source, let me know. Regards, Paco |
Attachment: 18157382171.txt
| | |
|
John Mitchell Posts:3471


| | 08/15/2008 2:44 PM |
| Yes, that is the source I was looking for. The replacement rule works in my debugger so I have no idea why it would not in your site.
I'll see if I can still get into your server with the info you gave me in PM before. | | | |
|
paco
 Posts:27

| | 08/17/2008 11:04 AM |
| Hello John, The source file that I sent you was from the production envrionment and the rule was commented off. The behaviour remains the same on production as on the test server. Maybe you wanted the source file with the rule on? I attached it here for you. If you want to take a look at this one, maybe you can find something then? You can always take a look on the test server. The login credentials are the same as before and you still have rights. Don't forget to turn the rule on then! Regards, Paco |
Attachment: 18174590571.txt
| | |
|
John Mitchell Posts:3471


| | 08/22/2008 8:11 AM |
| Ok, sorry for the delay. I logged in and copy and pasted the viewstate rule from the default.config and it is working now. There must have been a format problem with your config file. | | | |
|
paco
 Posts:27

| | 08/22/2008 10:37 AM |
| Hello John, Thanks for solving the issue. I did not know what to look for and thought there was something more going on. Next time I will try first to copy/paste a second time ;-) I still have the following issue: Because of using the Aggregator module from DnnStuff in combination with Pageblaster I had to change the rule "Move dnn js links to Head". I had to set the "repeatTimes" to "1". The error I got was a javascript error saying 'DNN' is undefined. Maybe you can take a look at this too? I left the setting on the test server to "99" so you can see the error. To see the error you can go to e.g. the page "website proposal" where the Aggregator module is being used. It is a javacript error which can be seen in the left bottom corner and can be opened for more details. Thanks in advance. Regards, Paco | | | |
|
John Mitchell Posts:3471


| | 08/22/2008 11:19 AM |
| | The Javascript error probably can not be avoided. I created those rules so that they only work when logged out because you get the same kind of error when logged in. It is because of namespace collisions, (two functions with the same name in different JS files), in the javascript when the other JS used for the action menu is loaded. The aggregator module must have the same problem. | | | |
|
paco
 Posts:27

| | 08/23/2008 4:05 PM |
| Hello John, I do not experience any problems on the page except for the js error message. The Aggregator seems to be working, when I change tabs or localization, it works fine. For the Pageblaster I do not know exact what the consequences are, maybe you can tell? If I look at the source it seems ok to me. What do you advice here?
Shall I change the node "repeatTimes" of the rule "Move dnn js links to Head" to "1" so the js error will not be there?
Or, if everything seems to be working, leave the node "repeatTimes" to "99" and take the js error for granted in favour of speed? I guess here that this rule is also made to make the webpage show faster, right? Thanks in advance. Regards, Paco | | | |
|
John Mitchell Posts:3471


| | 08/23/2008 8:03 PM |
| Yes, the rule is made to allow the javascript to be optimized, but overall that one rule isn't going to take a lot away from performance you might have had if everything worked correctly.
Personally, I would opt for making sure it works, (even if it is just a JS error that doesn't manifest itself in a visible error), over any performance gain.
What exactly is the error? | | | |
|
paco
 Posts:27

| | 08/24/2008 2:47 AM |
| Hello John, This is the error: Line: 25
Char: 1
Error: 'dnn' is undefined
Code: 0 This is line 25: document.getElementById('dnn_ctr492_Aggregator_selected').value=tabname.id; This is line 26: dnn.dom.setCookie('492_selected', escape(tabname.id), null, '/');
This one seems like causing the js error instead of line 25. I copied the source and attached it here. Regards, Paco |
Attachment: 18244764771.txt
| | |
|