ISSUE-209: Encode keys before storing on mongo.

[samt2497]

Issue: #209

Notes:

• Added encoding on the profile ( replace . with it Unicode equivalent )

[markstory]

Looks good. I will get the build fixed for php7.

[lauripiisang]

It worries me that for me, everything works with mongodb 3.0 (see issue #209 ).

I would like to set up a docker env to test this and find out what the real cause of this issue is before we hurry to a fix that might not work. If you need to use your fix, you can do so via https://getcomposer.org/doc/05-repositories.md#loading-a-package-from-a-vcs-repository

[lauripiisang]

why is this recursive call? do we have any other problematic keys than the content of array_keys($profile['profile'])?

[lauripiisang]

Are there any nested structures inside a profile that I have ignored?

[lauripiisang]

If this is the case that we do not need recursive checking, we should instead opt for an easier solution.

What do you think about something like

public function encode($profile) {
  // return $profile if the array is empty or not an array
  $replacedKeys = str_replace('.', '．', array_keys($profile));
  return array_combine($replacedKeys, $profile);
}

?

Here's a demonstration, if you're intersted

<?php
$a = ['fx' => ['a' => 'xxx', 'b' => 2, 'c' => 3], 'quux' => ['f' => 7, 'cx' => 3]];
print_r(array_combine(str_replace('x', 'y', array_keys($a)), $a));

prints

Array
(
    [fy] => Array
        (
            [a] => xxx
            [b] => 2
            [c] => 3
        )

    [quuy] => Array
        (
            [f] => 7
            [cx] => 3
        )

)

[lauripiisang]

I'd also recommend defining the . and ． as constants (UNICODE_PERIOD and UNICODE_FULLWIDTH_PERIOD for example?)

[glensc]

Superseded by #352