rtcamp / rt-lib Goto Github PK

We need to use composer - https://rtcamp.com/tutorials/composer-wordpress/

It will be good to have every class as separate composer package in future.

https://travis-ci.org/rtCamp/rt-lib

If Travis supports PHP 5.5 test and WordPress beta releases (3.9 beta for example), it would be nice.

Ref : https://github.com/rtCamp/rt-lib/blob/develop/rt-user-groups/class-rt-user-group.php#L22

No need to use realpath(__DIR__.'/../').'/ because we can use relative path and __DIR__ introduced in php 5.4, WordPress support php 5.2.4 - So it is throwing error for user who are on php 5.3 or older version

__DIR__ is actually equivalent to realpath(dirname(__FILE__)) , To ensure the DIR constant is defined for PHP 4.0.6 and newer use following snippet
(@__DIR__ == '__DIR__') && define('__DIR__', realpath(dirname(__FILE__)));

See: http://momentjs.com/

In many project we need relative dates.

I am not sure if this will make sense as a PHP class!

https://github.com/rtCamp/rt-lib/blob/master/wp-helpers.php#L27

Here the constant is defined without the check of defined() which is resulting in a PHP Notice.

Some good examples here: http://stackoverflow.com/a/2193937/156336
Also https://github.com/willdurand/EmailReplyParser

We can polish our internal CRM library as well and make it available for reuse.

• Private Attributes : This remains only in WordPress Back-end for admin use purpose only
• Public Attributes : By default all attributes will be public.

• Change Label to Mailbox : Ref
• Enqueue script/style on proper hooks : Ref
• Add filter for tabs. Someone would want to add custom tabs : Ref
• Add action for tab content in reference to above custom tabs : Ref
• Move all such independent functions to single file. You may name it rt-mailbox-functions.php : Ref
• Move out this class. It is no longer a helper class in rtMailbox context : Ref
• Remove this debug line : Ref
• Rename this folder to settings. Those classes are not templates : Ref

From PHP 5 onwards, all objects are passed by reference, so &$this is redundant. This is a common pattern with add_action/add_filter calls (since the callback is an array), but should just the bare $this instead. Read More