Skip to content

Commit

Permalink
Fixed strange error where some servers gave a 406 Unacceptable error …
Browse files Browse the repository at this point in the history
…on any file that had "cookie" in the name.
  • Loading branch information
jerel committed Jun 3, 2011
1 parent 93e6b54 commit 6cef90f
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion system/pyrocms/modules/files/controllers/admin.php
Expand Up @@ -116,7 +116,7 @@ public function index()
->title($this->module_details['name'])
->append_metadata( css('jquery.fileupload-ui.css', 'files') )
->append_metadata( css('files.css', 'files') )
->append_metadata( js('jquery/jquery.cookie.js') )
->append_metadata( js('jquery/jquery.cooki.js') )
->append_metadata( js('jquery.fileupload.js', 'files') )
->append_metadata( js('jquery.fileupload-ui.js', 'files') )
->append_metadata( js('jquery.ba-hashchange.min.js', 'files') )
Expand Down
2 changes: 1 addition & 1 deletion system/pyrocms/modules/navigation/controllers/admin.php
Expand Up @@ -138,7 +138,7 @@ public function index()
// Create the layout
$this->template
->append_metadata( js('jquery/jquery.ui.nestedSortable.js') )
->append_metadata( js('jquery/jquery.cookie.js') )
->append_metadata( js('jquery/jquery.cooki.js') )
->title($this->module_details['name'])
->build('admin/index', $this->data);
}
Expand Down
2 changes: 1 addition & 1 deletion system/pyrocms/modules/pages/controllers/admin.php
Expand Up @@ -142,7 +142,7 @@ public function index()
$this->template
->title($this->module_details['name'])
->append_metadata( js('jquery/jquery.ui.nestedSortable.js') )
->append_metadata( js('jquery/jquery.cookie.js') )
->append_metadata( js('jquery/jquery.cooki.js') )
->append_metadata( js('index.js', 'pages') )
->append_metadata( css('index.css', 'pages') )
->build('admin/index', $this->data);
Expand Down

7 comments on commit 6cef90f

@marcoscoelho
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol..

@yvonn
Copy link

@yvonn yvonn commented on 6cef90f Jun 4, 2011

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed it - but now I get another error

http://bit.ly/mKnnqY

@yvonn
Copy link

@yvonn yvonn commented on 6cef90f Jun 4, 2011

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just did it, but new error instead
http://bit.ly/mKnnqY

@jerel
Copy link
Member Author

@jerel jerel commented on 6cef90f Jun 6, 2011

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yvonn It would appear as if you created an error in the tree_builder() method in the pages admin controller. If you want to paste that file at pastie.org or similar and make a link to it I'll look at it for you.

@yvonn
Copy link

@yvonn yvonn commented on 6cef90f Jun 10, 2011

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

edit

@yvonn
Copy link

@yvonn yvonn commented on 6cef90f Jun 26, 2011

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Jerel,
Can you tell me what have to be changed at the hosting side (my provider). Because if this is an server config issue at my providers server then maybe I can ask them to add something??
I tried to add sub pages on my local machine and that works fine.

@jerel
Copy link
Member Author

@jerel jerel commented on 6cef90f Jun 27, 2011

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renaming jquery.cookie.js to jquery.cooki.js as done in this commit fixes it.

The issues stems from servers not allowing any file with "cookie" in the name. So if you can have your provider set the server to start allowing file names with "cookie" in them then that will fix it too.

Please sign in to comment.