Changeset 1164 for trunk/common/db/attachment.php

Timestamp:

01/14/12 16:18:10 (4 months ago)

Author:

bogdan2412

Message:

Make infoarena compatible with HPHP.

• Implemented APC caching support since eaccelerator is not compatible with HPHP and although we support memcached, we do not use it on live.
• Replaced instances of create_function_cached with php anonymous functions. This bumps the PHP version requirement to 5.3, but it allows the code to work with HPHP, since dynamic function generation is replaced with staticly analyzable functions. This also removes a horrible, terrible hack in Textile.php which was storing the current instance of the Textile class being processed as a static variable so that it could be accessed by anonymous functions.
• Replaced occurrences of assert with log_assert.
• When running under HPHP, IA_HPHP_ENV is defined and is used to bypass some incompatible code (redundant ini configuration checks).
• Fixed some includes that were dependent on the working directory being www/
• Included a sample HPHP configuration file and a Makefile with compilation commands.

REVIEW URL: ​http://reviewboard.infoarena.ro/r/182/

File:

• trunk/common/db/attachment.php (modified) (3 diffs)

trunk/common/db/attachment.php

@@ -151 +151 @@
 // You may use % as a wildcard
 function attachment_get_all($page, $name='%', $start = 0, $count = 99999999) {
-    assert(is_whole_number($start));
-    assert(is_whole_number($count));
+    log_assert(is_whole_number($start));
+    log_assert(is_whole_number($count));
     $query = sprintf("SELECT ia_file.*, ia_user.username, ia_user.full_name as user_fullname
                       FROM ia_file
@@ -165 +165 @@
 // _count for the above.
 function attachment_get_count($page, $name='%', $start = 0, $count = 99999999) {
-    assert(is_whole_number($start));
-    assert(is_whole_number($count));
+    log_assert(is_whole_number($start));
+    log_assert(is_whole_number($count));
     $query = sprintf("SELECT COUNT(*)
                       FROM ia_file
@@ -184 +184 @@
 // FIXME: does this belong here?
 function attachment_get_filepath($attach) {
-    assert(is_array($attach));
+    log_assert(is_array($attach));
     return IA_ROOT_DIR.'attach/'.
             strtolower(preg_replace('/[^a-z0-9\.\-_]/i', '_', $attach['page'])) . '_' .