Merge remote branch 'origin/core/1.1.1/maint' into core/master

Fix check.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Fix invalid error detection.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

In case of RPC error, show the HTTP answer.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Produce errors in case of RPC failures.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Remove debugging stuff.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add an embedded mode for platal.

You can query /embedded/{mode}/path/to/url to get a mode in which the
skinning mode information is propagated through pages.

e.g.: https://www.p.org/embedded/light/search can be used to get the
search page with light skinning.

Note: This only requires a hack in .htaccess to work with static files.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

PlImage supports timestamps.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Allow insertion of a batch of entries.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Ensure PlBacktrace is loaded before registering errors.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Fix permission check for page edition.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Take into account PlExportable interface.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Coding rules.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add an helper to build the export of the PFOs.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Removed unconsistent reference passing in PlFilter

Instances are already passed by reference in PHP. Adding & only obfuscates the
code and make PHP throw notices.

Signed-off-by: Riton <riton@melix.net>
Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

PlFilterOrder is an abstract class, put a default implementation for
export() there.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

PlFlagset and PlDBTableEntry are exportable.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Added interface PlExportable and implemented it on PlFilter, PlFilterCondition and PlFilterOrder

Classes which implement PlExportable must expose an export() method.
It should return an associative array representing the instance.

Signed-off-by: Riton <riton@melix.net>
Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

PlDBTableEntry: add support for field validators.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Do not show old errors when clearing them.

Signed-off-by: Stéphane Jacob <sj@m4x.org>

Prevents error when displaying site errors.

Signed-off-by: Stéphane Jacob <sj@m4x.org>

Fix some php errors.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Several improvements on how PHP errors are managed.

Use PlBacktrace to handle PHP error instead of $GLOBAL['pl_errors']. When
output is in json add a special key with the traces. When site is in debug
mode, errors are automatically dumped by PlBacktrace. When site is in
production mode, use the legacy pl_print_errors to print the errors at the
bottom on the page.

This commit also introduce a new solution to build json handlers: if the
handler return PL_JSON, then the site will produce json from the variables
previously assigned vi jsonAssign.

Note:  Don't use jsonDisplay() for the new json output because
jsonDisplay() output contains user-controlled content (I mean, content
passed by the user through Post, Get or Cookie) and this is not a good
idea: the site should never output something it does not fully control.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Clear cache link in menu also clear PlCache.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add PlCache::clear{Variant}() and PlCache::clearAll().

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

PlDBTableEntry: can provide a custom formatter to serialize/unserialize
the value of a field.

Also provide a JSon formatter.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add can_convert_to_integer and to_integer functions.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

PlDBTableEntry: add support for custom selection.

Selection based on a non-primary key is now possible.
 $user = new PlDBTableEntry('accounts');
 $user->uid = 26071;
 $user->fetch(); /* selection on primary key */

 $user = new PlDBTableEntry('accounts');
 $user->hruid = 'florent.bruneau.2003';
 $user->fetch(); /* selection on non-primary unique key */

This also works for multiple keys in case of iteration
  $selector = new PlDBTableEntry('accounts');
  $selector->type = 'ax';
  foreach ($selector as $user) {
     $user->comment = 'AX secretary';
     $user->update();
  }

PlDBTableEntry now expose iterateOnCondition that takes a custom condition
for the entry selection.
  $selector = new PlDBTableEntry('accounts');
  $it = $selector->iterateOnCondition('FIND_IN_SET(\'watch\', flags');
  while ($user = $it->next()) {
    $user->comment = 'Watched account';
    $user->update();
  }

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

PlDBTableEntry: add support for date(time) fields, insertion and deletion.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Security...

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Should avoid fatal errors when sending emails from the website...
(including automatic emails).

Don't know how we didn't notice this one before.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Release core 1.1.1

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Fix select_db_table to handle properly 'valeur=0' commands

empty() is an evil PHP function:
    empty($a['x']) is true if $a['x'] == 0...

Signed-off-by: Raphaël Barrois <raphael.barrois@polytechnique.org>

Add a new class that aims at providing a simple gateway between stored
data and php code when data are fetched 'as is'.

UseCase: classes like Phones or Addresses.

How to use:
 /* Fetching the content of a entry */
 $entry = new PlDBTableEntry('accounts', true /* autofetch */);
 $entry->uid = 26071;
 print $entry->hruid;

 /* Iterating on entries and saving changes*/
 $selector = new PlDBTableEntry('profile_phones');
 $selector->pid = 26071;
 $selector->link_type = 'pro';
 foreach ($selector as $entry) {
    $entry->comment = 'Professional phone number';
    $entry->save();
 }

Note: ATM, SQL joins are not supported, but this is planned for the
future with an API that might looks like:
  $entry = new PlDBTableEntry(array($mainTable,
                                    PlSqlJoin::left(),
                                    PlSqlJoin::inner()));

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add an API to handle caching.

In most cases, the API usages is the following:
  $var = PlCache::get{Variant}('key',
                               'factory',
                               array('factoryarg1','factoryarg2', ...));

PlCache::invalidate{Variant}, set{Variant} and has{Variant} are also
provided.

The Variant provides hints on the expiration of the data and whether it
can be shared between users/sessions or not. Available variants are:
 - Local = no sharing, invalidate data after the execution of the page
 - Session = no sharing, invalidate data when the session dies
 - Global = sharable, can invalidate data after a timeout

Examples:
  function buildDataFromDb($tableName)
  {
     return do_something_very_expensive($tableName);
  }

  /* Example using all-in-one 'get' variant' */
  $value = PlCache::getSession('blah_' . $tableName',
                              'buildDataFromDb',
                              array($tableName),
                              1 /* ensure the value is not
                                   computed more than once
                                   per second */
                              );

  /* Same example using has, set and get */
  if (!PlCache::hasSession('blah_' . $tableName)) {
    $value = buildDataFromDb($tableName);
    PlCache::setSession('blah_' . $tableName, $value, 1);
  } else {
    $value = PlCache::getSession('blah_' . $tableName);
  }

Note: this second example can in very specific conditions produce invalid
results if hasSession is called before expiration of the data and
getSession after.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Returns transaction result.

Signed-off-by: Stéphane Jacob <sj@m4x.org>

Improves site_errors.

The new version include the context of the error and is easily extensible.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Use explicit click on the trace to toggle its visibility.

Kill the long standing "Hover me pour la trace". This was pretty cool a
few years ago, but it tends not to work as well as it used to do with
modern browsers.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Update ChangeLog

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add a guard against overlapping transactions.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Fix deprecated references to XorgDBIterator (renamed to XDBIterator).

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add XDB::runTransaction($callback, $arg1, $arg2, $arg2...) to run a
callback as a SQL transaction with automatic commit/rollback in case of
success/failure.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add XDB::startTransaction(), XDB::commit() and XDB::rollback().

Note: you should always use these functions when using transactions since
they perform the necessary post/pre actions to ensure the environment is
well configured.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Fix SUID.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add unit tests for soundex.

The goal of this tests is to provide an improved version of the soundex
algorithm in the future.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Don't be so lazy, implements all the XDB::raw* variants.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Use XDB::raw variants.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add XDB::rawXXXX($query) that runs the given query without formatting.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

TableEditor: don't use REPLACE to update the content of a row.

REPLACE deletes the row and then insert the new value. If the content of
the row is used as a foreign key by another table, this leads to a full
cleanup of all the entries that point to the deleted row... and when you
update the 'x' account_type, this removes all the accounts.

This also fix some minor programming errors:
 * let XDB perform the escaping, don't use addslashes manually.
 * don't call XDB('machin = {?} AND ' . $someStringContainingUserInput, $myvalue)
  (this is not critical since the placeholder is before the
  'random' string, but this can lead to errors if the 'random' string
  contain a placeholder {?}).

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add PlUser::isMe($other): returns true if $other is a reference to the
same user.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Use user object from session to compute permission when using 'hasPerm' in
templates.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

PlFlagSet is iterable.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add interface PlIteraface and the corresponding abstract class
PlAbstractIterable.

PlIterable aims at providing a common interface for the objects that can
produce a PlIterator. It extends PHP's IteratorAggregate that provides an
interface for producing a PHP's iterator.

PlAbstractIterable provides a default implementation for
IteratorAggregate::getIterator.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add tests for XDBFormat.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Add PlIteratorUtils::foreachIterator() to build a PHP's Iterator from a
PlIterator.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Directly invoke XDB::prepare() instead of using __autoload + call_user_func_array.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Big cleanup of XDB.

 * get rid of these annoying _ in member and method names
 * introduces XDBFormat interface in order to allow objects to provide
   their own formatting when used as a query argument.
 * s/XorgDB/XDB/

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Extracted PlSqlJoin out of PlFilter class file

Signed-off-by: Riton <henri.jouhaud@polytechnique.org>
Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Forgot to switch the constructor to protected

Signed-off-by: Riton <henri.jouhaud@polytechnique.org>
Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Prevents warnings in plwizard.tpl.

Signed-off-by: Stéphane Jacob <sj@m4x.org>

Don't allow empty flags.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Merge remote branch 'origin/core-1.1.0' into core

Actual security fix id.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Fix an error in permission handling: all pages where accessible without
admin permission.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Prevents php warning when trying to retrieve a non-existing image (Closes #1146).

Signed-off-by: Stéphane Jacob <sj@m4x.org>

Update ChangeLog

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Made plat/al php5.3 compliant

- split => preg_split or explode
- handler(&$page) => handler($page)
- abstract static => interface

Signed-off-by: Riton <henri.jouhaud@polytechnique.org>
Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Switched PlImage's fields from private to protected

Signed-off-by: Riton <henri.jouhaud@polytechnique.org>
Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Inserted a pl_error field in the json responses

Signed-off-by: Riton <henri.jouhaud@polytechnique.org>
Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Minor fix on PlUser: declare the "email" field too.

Signed-off-by: Raphaël Barrois <raphael.barrois@polytechnique.org>

Define a range of debug flags available for "users" (ie clients of the
core).

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Oops.

Signed-off-by: Raphaël Barrois <raphael.barrois@polytechnique.org>

Add "groups" to PlFilter.

Would return the list of (name, nb_res) for a given field

Signed-off-by: Raphaël Barrois <raphael.barrois@polytechnique.org>

Update doc for PlSet to explain 'arg' functions

Rename the Smarty part of these for consistency.

Signed-off-by: Raphaël Barrois <raphael.barrois@polytechnique.org>

Add comments to plset.php, remove useless var from calls.

Signed-off-by: Raphaël Barrois <raphael.barrois@polytechnique.org>

Removes trailing spaces.

Signed-off-by: Stéphane Jacob <sj@m4x.org>

Add a $page-> function for displaying a message and a "continue" link

For instance, "The following users have been added correctly; continue?"

Signed-off-by: Raphaël Barrois <raphael.barrois@polytechnique.org>

Slightly modifies select_db_table plugin.

Signed-off-by: Stéphane Jacob <sj@m4x.org>

Fixes use of an uninitialised variable in icon compiler.

Signed-off-by: Stéphane Jacob <sj@m4x.org>

Prevents notice in skin templates.

Signed-off-by: Stéphane Jacob <sj@m4x.org>

Manage site errors using exceptions.

This has 3 main avantages:
 - Error may not be fatal (e.g. a SQL error must not be fatal during the
   execution of the unit tests)
 - We can have the backtrace
 - The same code handle all the kind of errors.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Changelog for the next release.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Should fix Recoverable Errors on wiki pages.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Release time for the core.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Prenvents '##' to be escaped to '' in SQL querries (Closes #1156).

Signed-off-by: Stéphane Jacob <sj@m4x.org>

Enables sorting of option fields in table editor.

Fix table editor : edit entry with id = 0 doesn't mean add a new entry

Add PlMailer::setTo() (thus making PlMailer reusable).

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

When force_login doesn't work, auth will fail

MMList : adds a method to replace email in all lists where it is used

Try not to return empty email if you can find one

small hack to return by ref

Add optional fields listing from other tables in table editor.

Update ChangeLog

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Should fix a long standing bug preventing mail with accentuated From to be
sent and generating corrupted To/Cc headers. Close #1073.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Fixes message when adding a new entry with tableeditor. Closes #1022.

Signed-off-by: Pascal Corpet <pascal.corpet@m4x.org>

Implements __unset on PlUser.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

Fix quoted-printable encoder when the subject contains a '?'

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>

fix HTML validation

fix toggledesc in PlFilterOrder. Fixes all order bugs in PlSets