jgen
50686ac61e
More linting, fix PHPDoc style.
2014-04-24 22:29:29 -04:00
jgen
daee99c1bc
Removing the closing PHP "?>" tag as per style guidelines.
2014-04-24 22:28:53 -04:00
jgen
860af25550
Need to pull in the global $user variable.
2014-04-24 22:22:45 -04:00
jgen
4ec5a7ce52
Removing the closing PHP "?>" tag as per style guidelines.
2014-04-24 22:22:16 -04:00
jgen
780d5f8709
More linting. Explicitly declare these class variables, as well as comment out dead code.
2014-04-24 22:13:00 -04:00
jgen
6e4ae229ef
Explicitly declare this class variable
2014-04-24 19:13:41 -04:00
jgen
b3077d5bcd
Removing the closing PHP "?>" tag as per style guidelines.
2014-04-24 19:13:20 -04:00
jgen
2706f72ce4
Update and fix the PHPDoc comments.
2014-04-24 19:08:23 -04:00
jgen
4cc31df737
Update the comments to PHPDoc style guidelines.
2014-04-24 19:02:43 -04:00
jgen
ab3f9850e4
Removing the closing PHP "?>" tag as per style guidelines.
2014-04-24 19:01:47 -04:00
jgen
286e75bef0
Scrutinizer complains that these functions aren't defined, yet they really are.
2014-04-24 18:45:54 -04:00
jgen
709366b70c
Explicitly declare these class variables.
2014-04-24 18:44:46 -04:00
jgen
7b90331f02
Add missing semi-colon.
2014-04-24 16:56:04 -04:00
jgen
993aacaea5
Update comment to reflect that a bug has been filed with PHP-analyzer.
2014-04-24 16:55:34 -04:00
jgen
c4dcba1e91
Explicitly check for the existence of these properties before attempting to use them.
2014-04-24 05:37:26 -04:00
jgen
7e41194b7f
Shish you are too clever for scrutinizer-ci.
2014-04-24 05:29:02 -04:00
jgen
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
jgen
e2f5c7a6ec
Check for null, just in case.
2014-04-24 05:09:38 -04:00
jgen
0501e05cff
Comment out dead code/vars.
2014-04-24 05:09:16 -04:00
jgen
332c8538b7
Change the function syntax to quiet JSHint.
2014-04-24 04:46:23 -04:00
jgen
868f8c9225
Commenting out dead variables and dead code.
2014-04-24 04:44:28 -04:00
jgen
f9c8a1720e
Comment out dead variables.
2014-04-24 04:41:13 -04:00
jgen
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
jgen
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
jgen
dbe30ad37d
Rename variable to make it more clear.
2014-04-24 01:36:30 -04:00
jgen
afc01fda5f
Prevent scrutinizer-ci from complaining about these fall-thoughs.
2014-04-24 01:36:04 -04:00
jgen
506e699f8a
Not sure if we really need the triple equals.
2014-04-24 01:15:30 -04:00
jgen
866d39a68e
The 'tags' box actually has a different ID.
2014-04-24 01:15:03 -04:00
jgen
51c145d5c9
Some more linting on the bookmarklet.js file.
2014-04-24 01:08:47 -04:00
jgen
4b90faf89d
Even more linting of JS files.
2014-04-24 00:55:14 -04:00
jgen
a105090039
Some more JS linting.
2014-04-23 23:39:09 -04:00
jgen
887f963e17
Add the jshint comment to alter how scrutinzer-ci deals with the js.
2014-04-23 23:38:53 -04:00
jgen
31501e48fe
More JS linting.
2014-04-23 23:31:20 -04:00
jgen
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
jgen
b775c007cd
Give these HTML headers some slight priority over the default value of 50.
2014-04-23 23:07:10 -04:00
jgen
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
jgen
8cdf919827
More linting. This time of the JavaScript for the chat-box extension.
2014-04-23 22:43:42 -04:00
jgen
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
jgen
5eaa637bbb
Merge branch 'develop' of git://github.com/shish/shimmie2 into develop
2014-04-23 22:23:53 -04:00
Shish
44d0e56c97
Merge pull request #406 from jgen/video_thumb
...
Video thumbnails
2014-04-20 11:25:24 +01:00
Shish
ef3daaf388
Merge pull request #411 from jgen/develop
...
Chatbox is broken on develop branch
2014-04-20 11:21:43 +01:00
jgen
21e1359b86
Avoid short open tag for PHP.
2014-04-20 05:07:26 -04:00
jgen
60980c8839
One more jQuery reference.
2014-04-20 05:04:45 -04:00
jgen
a7ba0f887b
Again, the chatbox needs help from the jQuery migrate plugin.
2014-04-20 05:00:59 -04:00
jgen
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
jgen
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
jgen
880840623a
Need to ensure that this html header is added last.
2014-04-20 04:36:25 -04:00
jgen
717401ccab
Removing this as jQuery is already included in Shimmie now.
2014-04-20 04:35:33 -04:00
jgen
8b698e1b6f
Revert this commit.
2014-04-20 04:29:11 -04:00
jgen
54362a49c6
Fix the broken chatbox issue.
2014-04-20 04:25:08 -04:00
