860af25550
Need to pull in the global $user variable.
2014-04-24 22:22:45 -04:00
4ec5a7ce52
Removing the closing PHP "?>" tag as per style guidelines.
2014-04-24 22:22:16 -04:00
780d5f8709
More linting. Explicitly declare these class variables, as well as comment out dead code.
2014-04-24 22:13:00 -04:00
6e4ae229ef
Explicitly declare this class variable
2014-04-24 19:13:41 -04:00
b3077d5bcd
Removing the closing PHP "?>" tag as per style guidelines.
2014-04-24 19:13:20 -04:00
2706f72ce4
Update and fix the PHPDoc comments.
2014-04-24 19:08:23 -04:00
4cc31df737
Update the comments to PHPDoc style guidelines.
2014-04-24 19:02:43 -04:00
ab3f9850e4
Removing the closing PHP "?>" tag as per style guidelines.
2014-04-24 19:01:47 -04:00
286e75bef0
Scrutinizer complains that these functions aren't defined, yet they really are.
2014-04-24 18:45:54 -04:00
709366b70c
Explicitly declare these class variables.
2014-04-24 18:44:46 -04:00
7b90331f02
Add missing semi-colon.
2014-04-24 16:56:04 -04:00
993aacaea5
Update comment to reflect that a bug has been filed with PHP-analyzer.
2014-04-24 16:55:34 -04:00
c4dcba1e91
Explicitly check for the existence of these properties before attempting to use them.
2014-04-24 05:37:26 -04:00
7e41194b7f
Shish you are too clever for scrutinizer-ci.
2014-04-24 05:29:02 -04:00
ce524e8729
This variable does exist, it is actually pulled from the database. Making it explicit for the linter.
2014-04-24 05:25:12 -04:00
e2f5c7a6ec
Check for null, just in case.
2014-04-24 05:09:38 -04:00
0501e05cff
Comment out dead code/vars.
2014-04-24 05:09:16 -04:00
332c8538b7
Change the function syntax to quiet JSHint.
2014-04-24 04:46:23 -04:00
868f8c9225
Commenting out dead variables and dead code.
2014-04-24 04:44:28 -04:00
f9c8a1720e
Comment out dead variables.
2014-04-24 04:41:13 -04:00
f67220f64b
From stack-overflow: The 'var' keyword is for declaring class variables in PHP4. It works in PHP5 but can raise an E_STRICT warning in from version 5.0.0 up to version 5.1.2, as it has been deprecated.
...
Changing to public as that is really what these variables are anyways.
2014-04-24 04:36:05 -04:00
67c087c30f
From stack-overflow: The 'var' keyword is for declaring class variables in PHP4. It works in PHP5 but can raise an E_STRICT warning in from version 5.0.0 up to version 5.1.2, as it has been deprecated.
...
Changing to public as that is really what these variables are anyways.
2014-04-24 04:30:58 -04:00
dbe30ad37d
Rename variable to make it more clear.
2014-04-24 01:36:30 -04:00
afc01fda5f
Prevent scrutinizer-ci from complaining about these fall-thoughs.
2014-04-24 01:36:04 -04:00
506e699f8a
Not sure if we really need the triple equals.
2014-04-24 01:15:30 -04:00
866d39a68e
The 'tags' box actually has a different ID.
2014-04-24 01:15:03 -04:00
51c145d5c9
Some more linting on the bookmarklet.js file.
2014-04-24 01:08:47 -04:00
4b90faf89d
Even more linting of JS files.
2014-04-24 00:55:14 -04:00
a105090039
Some more JS linting.
2014-04-23 23:39:09 -04:00
887f963e17
Add the jshint comment to alter how scrutinzer-ci deals with the js.
2014-04-23 23:38:53 -04:00
31501e48fe
More JS linting.
2014-04-23 23:31:20 -04:00
2dc6e054c8
These sorts are necessary on the header arrays, otherwise the relative priority of certain headers is not respected.
2014-04-23 23:28:05 -04:00
b775c007cd
Give these HTML headers some slight priority over the default value of 50.
2014-04-23 23:07:10 -04:00
a0ab3864c0
Push the headers for the chatbox even further down to make sure that jQuery is already loaded.
2014-04-23 23:01:51 -04:00
8cdf919827
More linting. This time of the JavaScript for the chat-box extension.
2014-04-23 22:43:42 -04:00
0bd031a269
Merge branch 'develop' of github.com:jgen/shimmie2 into linting
...
Manually resolved conflicts:
ext/chatbox/js/yshout.js
2014-04-23 22:42:30 -04:00
5eaa637bbb
Merge branch 'develop' of git://github.com/shish/shimmie2 into develop
2014-04-23 22:23:53 -04:00
44d0e56c97
Merge pull request #406 from jgen/video_thumb
...
Video thumbnails
2014-04-20 11:25:24 +01:00
ef3daaf388
Merge pull request #411 from jgen/develop
...
Chatbox is broken on develop branch
2014-04-20 11:21:43 +01:00
21e1359b86
Avoid short open tag for PHP.
2014-04-20 05:07:26 -04:00
60980c8839
One more jQuery reference.
2014-04-20 05:04:45 -04:00
a7ba0f887b
Again, the chatbox needs help from the jQuery migrate plugin.
2014-04-20 05:00:59 -04:00
2df54b911a
Don't use short open tags for PHP as they are not turned on by default in production environments.
2014-04-20 05:00:34 -04:00
e2516fa5cf
Chatbox should be fixed now.
...
The chatbox extension actually needs some functionality that was removed from jQuery versions 1.9 and up. When Shimmie's version of jQuery was bumped up to 1.11 this actually broke the chatbox.
2014-04-20 04:51:19 -04:00
880840623a
Need to ensure that this html header is added last.
2014-04-20 04:36:25 -04:00
717401ccab
Removing this as jQuery is already included in Shimmie now.
2014-04-20 04:35:33 -04:00
8b698e1b6f
Revert this commit.
2014-04-20 04:29:11 -04:00
54362a49c6
Fix the broken chatbox issue.
2014-04-20 04:25:08 -04:00
57a390eaab
Massive linting!
2014-04-19 23:38:42 -04:00
ff722906b7
Holy shit, this is really really nasty old JS code. I would love to dump this if possible.
2014-04-19 23:06:48 -04:00
