labofsahil/github-event-dataset-2012 huggingface dataset:
@cordoval I mean that this query is wrong, we now update recommendations counter and hold this info in database, so no need for count in query.

But the migration script needs to be extended to "migrate" also old bundles, loop over and count + save actual recommendations.
> What if I want to use jQuery, do I need to wrap that too with Meteor.is_client ?

jQuery ships to the client by default, so no need to take care of this...

But in general, what i have seen so far, if a lib is intended to work on the client only, you have to wrap it with Meteor.is_client.

I personally don't see problem here... Meteor is "write once, run everywhere"... so if you don't wrap your code, it's everywhere... if you want it on the client only, wrap it with Meteor.is_client, or on the server only, wrap it with Meteor.is_server.
@stof checked with @alex88 and seems like the single test fails without it...
is there really a difference between
```ruby
return false unless role
``` and
```ruby
return false if role.nil?
```
CS alignment
actually, the first condition is not complete
Yes, it should be `e`. Actually I wanted to capture particular `KeyError` like `yt$state` later figured out you can't do it. It should be `e.key == 'key'`, will remove it.
Maybe check for os.environ('COLUMNS') here too? Since you add it the next time, or do you assume it may change during execution?
fixed in FSL branch (not sure why the FSL stuff is even in here)
I have to replace this by this to make it work :

```php
        'username'       => 'response.user.firstName',
        'displayname'    => 'response.user.lastName',
        'email'          => 'response.user.contact.email',
        'profilepicture' => 'response.user.photo',
```
Style: brace on the same line.
What I hear you saying is that the following code is what you would prefer?

```python
from phonebook.helpers import PERMISSION_LEVELS

self.privacy_dict = dict(
                basic_info_privacy=PERMISSION_LEVELS[0][0],
                contact_info_privacy=PERMISSION_LEVELS[0][0],
                tags_privacy=PERMISSION_LEVELS[0][0])
```

To me, this is more fragile but not a hill I will die on.
should be <pre>.byStart[ 1 ]</pre>
add whitespace padding
при выделении секций ("", ~, -) я руководствовался http://sphinx-doc.org/rest.html#sections
While I'm picking at nits, would you have the time to ts() the submit button and the landmark, too? :*
Actually, I don't think we need either.  You can just import Control.Error from the errors package.  It's really just a re-export of what's in the either package, but it allows us to avoid explicitly depending on either and having to concern ourselves with version bounds.
Should be chemical/x-mdl-molfile
Two nits:

1.  The "./" here is unnecessary, although it doesn't hurt.
2.  While this is the literal translation of the above pattern, it's needlessly inefficient. You could do

```javascript
var value = ZU.xpathText(metadata[0], './Type[1]');
if (value) {
````
Why not make this an FSM? This gives you onTransition stuff, external observability, if you want a rolling event log, …
good catch. there's no reason to remove the same document twice but we definitely need to execute the callback.

we should keep the promise and just add the `fn` to it.

`promise.addBack(fn)`
Видимо я руками добавил, а потом еще и гемка при сутановке добавила.
Since this test requires a running MongoDB instance to be available locally, it should be ignored. I'll be creating a JIRA shortly to use a single test that can use any BinaryStore instance, and to change our build so that we can specify the configuration of the BinaryStore (e.g., in a similar way to how we have different profiles for databases, we'll have different profiles for each repository configuration we want to test).
'string', not 'String', and m_ is unacceptable. Try this:

    public string Filename { get; private set; }
Documentation still incorrect
you should not use the base exception class but the exception of the package
This ```rescue``` statement should catch not only url parsing errors. Since here we have also parsing JSON and making request I guess we should catch such exceptions here as well. To distinguish these exceptions we can check ```e.message``` for example.
Автозамены - зло.
Very true. Code review is good :)
Yeah, the split is a little odd. Right now the "extension" handles all the public API methods that are exposed, whereas `Store` is more self-container.
Similarly we can change this function to use the subclass too.
Oh, wait, I just worked it out: you have a system with an imprecise clock, and submit at a rate greater than 1 message per "clock tick", so you need greater uniqueness than the tuple <time, thread> gives you.  You should probably document that.
Out of curiosity, what does just calling `pr-str` on each param give you? This is debug output, so it seems like it may be useful to output more interesting info for each parameter (such as its type, if necessary).
Wow, looks more like 10x slower to me :(
Please document the callback parameters
I'm really not a fan of these for (var foo in bar) type constructs.
When possible we should get the keys via Object.keys(members) and iterate over that array.
I'm wondering if we need the tenantoverride for individual fields. If we don't have a use case for it yet, it might be worth removing it for now.
We could redeclare the property and the constructor in our class.  That is a kludgy workaround for a private variable but it's been used before elsewhere in Symfony.
I think we can leverage the message on the CheckNotFound error, rather than writing our own message:

```
# from https://github.com/tstmedia/okcomputer/blob/master/lib/okcomputer/registry.rb#L15
raise CheckNotFound, "No check registered with '#{check_name}'"
```
When is this flag 'true'?
Please use `===` for CS ;)
Тут вы не правы, потому что Set гарантирует уникальность елементов и поэтому мутирован будет елемент единожды.
Реализация не зацикливается, но есть проблема определения эффективности алгоритма замены. Если ввести переменную iterationsCount и запустить мутацию для 5, 10, 50, 100%, то получим приблизительно такую картину:
5%: 5..11
10%: 10..24
50%: 80..111
100%: 438..1040
Можно добиться более линейной зависимости. Для 100000 записей и 100% замен получаем около 2 млн. итераций и тормоза.
Но в рамках задания все ок, это работает.
чтоб не было зацикливания поменяйте знак сравнения на строгий "<" 
 "&& " надо все же вернуть :)
"Создайте метод hammingDistance, который возвращает int и принимает объект класса Cell. Этот метод должен сравнивать свой ДНК и ДНК переданного в качестве аргумента объекта..." — если я правильно понимаю задание, предполагается что метод hammingDistance должен быть методом объекта, а не класса. Хотя это не принципиально.
При размере массива в 100 это работает правильно (100 / 100 = 1), но при другом размере массива, результат будет неправильным, так как в результате деления на 100 получится дробное число, которое будет округлено до ближайшего целого (int).
This needs to go.
`@iconPath = @config.stylus?.iconPath ? sysPath.join 'images', 'icons'`
Adding filter criteria is what you do in the FilterCollection when you combine filters. This is really just returning the combined criteria for the filter without modifying anything, so `getFilterCriteria()` seems more appropriate.

@asm89: Can you explain why you used "add" in the ORM SQLFilter class?
`.present?`  to return a true/false?
I would push this into the $.mobile.path object at the top of `jquery.mobile.navigation.js`.
What’s that for? I guess this is unused code which also causes the new PyLint warning.
This has nothing to do with SEO as we do not expose such an URL to the outside. It is just a shorter URL that people can use to refer to our resource pages. The loaded page then always takes care of redirecting to the long URL itself.

This is for example done by StackOverflow, too, see for example these two links to the same question:
* http://stackoverflow.com/questions/7041657/haystack-why-does-realtimesearchindex-sometimes-not-update-my-saved-object
* http://stackoverflow.com/questions/7041657/
why are we ignoring this exception?
Do you have any suggestion ?
``their``
从中文的角度上理解还好吧，"发出"显得更加怪异啊。。
Indentation here and on the next line.
@dummy.avatar.updated_at should just return @dummy.avatar_updated_at, whatever it is based on @dummy's life cycle.
I don't think we need this, as the drain listener that XNIO provides does basically the same thing.
其他地方也是
¡ESTO es simplificar! Ja, me gusta.
Good fix. A thouht, should we provide the basic template with a ```error``` variable that we can set to reflect errors in the controller/backend upon a request?
you should not wrap the return value inside parenthesis
Currently, if more than one of --zoom, --fit-width/height is specified, the "smallest one" will be used.
"--zoom disables --fit-every" is not intuitive to me, and not even written in the manpage.
task.cancel() ?
This whole function is a copy-paste from below. No changes.
could you add some javadoc to this method? It's not easy for me to understand what you're doing. javadoc is not meant only for those using it but also for those trying to understand the code.
Can't we cache a reference to the asyncStore locally rather than call a method and cast each time?
But, if you think uneven bin support is an absolute must then I could take a look at pcolorfast.
I'm not comfortable with the randomness at this step. I'd rather it be repeatable and picking the same version otherwise the test might become a bit more exploratory.
Even if just retaining the code we had previously (but remove the if statement as it's unnecessary) and adding a comment or making it more intuitive/readable.

However I'm OK with the randomness on line 77 though, in the context of the limitations of the test and its speed :)
It will be used in test_dialer.
Yup, will change
Interessante, nunca tinha tentado a lmodern. Ela parece mais uniforme, talvez um pouco mais fácil de ler. Pode ser que eu deixe a opção de ativá-la.
I would add a bit of descriptive text in the example to explain what * means (required field).
this needs to be in double quotes so that the variable apt::params::provider is interpolated
also, for clarity, I would be explicit about the namespace

"${::apt::params::provider} update"
Some EMI animations have duration 1000, with the parsing code multiplying that by 1000.
That leads to some scripts (ie. demo intro) to wait for 15 minutes for an animation to complete.
I'll refactor the code to keep the duration-1000 animations at duration 1000, which should make them finish in the same timespan.
Thanks for sparking the idea :)
it could use some spaces
Mind sending me that dataset?
What are the possible reasons that a user could not be created that would throw an error? Can that be handled better without a try/except?
oo good catch! yes.
- The assignment to ``$result`` seems to be dead. Are you sure it is necessary, or maybe you have a typo somewhere?
- Are you sure ``$params`` has the correct type as "null|array" is not a sub-type of "array"?
why are you removing this line since you don't remove it router.php?
there is a mime type guesser in Symfony2
Saying PANIC is redundant and I would remove the ?!?

Also please consider using log.Fatal instead, since the stacktrace isn't very user friendly.
What if tokenType is `nil`?
Why that space has been added?
Jus added the action to change theme
@gkossakowski: For most projects that should not be necessary. If it is necessary for some reason, then create eclipse.sbt in plugins/ and <project-dir>. Others can decide to exclude these files then.
and this patch should have been more focused, this is a completely separate thing, I just didn't read very well :D
They beautify of jQuery is that you can chain this together:

```
$('#notif_box').slideUp('fast').toggleClass('show');
```

But I am not sure why you are setting and clearing this class?
But in fact it would be much better if you would have such API (on the module-level):

````
get_horoscope_provider(language):
    """
    Returns an instance of the horoscope provider for requested language, or raises ``KeyError`` if it does not exist.
    """

def get_supported_languages():
    """
    Returns a list of supported languages (their language codes).

    That is, a list of language provided by defined horoscope providers.
    """ 
```
I think it is easier to understand the code and it looks better this way. :)
nodewatcher is not always the best source:

https://github.com/wlanslovenija/nodewatcher/blob/master/nodewatcher/frontend/templates/registration/activation_email_subject.txt

:-)
For one location.
You changed the logic here. If both `tmp_upper_limit` and `tmp_value` are `datetime.datetime`, then you do not have to convert anything!
This would more idiomatically be named `arith?`
If this isn't conditional on anything, then there isn't really a reason to have an args array, i.e. it these things can just be passed directly to configure.
I think you should fix the indentation to be consistent with the rest of the codebase.
I see no problems with that ;)
Help a noob here... should i make a second PR or is it possible to edit this one?
-> `(function ( window ) {`
Good point- I'm fine with leaving it as-is.
Add space after if, and remove spaces inside of parenthesis.
What's the deal with slf4j-simple? Don't we want to remove the implementation jar entirely? I think we only need slf4j-api.
I understand that cancelling a reservation and expiring a reservation do exactly the same thing... however, the business meaning of the two things is very different.  One means that a user cancelled the reservation while the other means that the system cancelled the reservation.  Is there a way in this form to tell the difference if we need to investigate a customer complaint?
Is this all the APIs, or are there more to come?
Add a check for the other params as well?
Good, confusing, and I didn't distinguish green and red parts well enough.
correctamundo :)
а, ну это конечно. но саму функцию стоит по другому назвать. store наверное лучше чем save
Arf, you mixed two pull request...
`name_ = ndb.ComputedProperty(lambda self: self.name.lower(), name='n_')`

or, if you want the long name in the datastore:

`name_ = ndb.ComputedProperty(lambda self: self.name.lower())`
Slightly neater to leave off the final position argument, since (0,0) is the default.
I think I'd prefer that this gets done on this branch, rather than merge as is.  These changes are also needed for the config stuff, so I don't know that there is any advantage to merging in two steps, and I'd rather not merge something that only passes the tests because sections of the tests were commented out.
+1 for form reuse ... also noted this field is in the model, then gets passed to client in exercise template JS object.
use ConfigKey for things like this?
See above comment on log...
I'd do a checkNotNull(is, "inputStream") on the first line. But not important (and presumably this is the same as the groovy code) so ignore...
is a smell, but can't think of something cleaner.  cf Collections2.
c'est 25% maintenant
git => hg
(and you'll go insane).
shouldn't this be `type.to_s.camelcase` ?
Needs the ordereddict library from PyPI if it's in 2.6. I'll add the dependency to setup.py
num?
Wouldn't that append the port? It only wants the IP here, port is separate.
Пакет должен присутствовать. Каждый пакет начинается со строчной буквы (надо переименовывать папки).
А почему ты удаляешь мои файлы?
При исключении в close() мы потеряем исходное сообщение об ошибке.
Не компилируется. Ты бы проверял перед отправкой.
Ну сделай уже отдельный класс IoUtils, в котором будет этот метод.
mit: left, right, right?
Cant this just be: `public SpellLoader loader = new SpellLoader(this);`
Reorder import?
s/creat/create
We should use assert contains here.
Am I running tests wrong somehow, because I'm not seeing failing tests regarding this.

In regards to the tests, I only changed instances of `getFormType` to `getInputClass` to match the new method name.  I'm getting 3 warnings in regards to the annotation "@FOS\RestBundle\Controller\Annotations\QueryParam" not existing - but I got those before I started working on this.
It's required by functions declared and called at module level. If there is a better alternative I'll implement that.
can T be more explicit? like n_times or any other more meaning full name?
You should delete that.
oops. assert_not_in. assert_not_in. assert_not_in.

!= assert_in.
bwj?
Its already done.
а треба? по ідеї, якщо не пусте поле, то вже ок
Can you make this logging nicer? The ## really don't do it for me.
Huh? I swear I fixed this a while ago, no?
https://github.com/jbossas/jboss-as/pull/1392
https://issues.jboss.org/browse/AS7-2099
Walter will get mad if you use tdata!
Hi! It should be `propel/propel1` :)
This method is documented to take 2 parameters and your implementation only supports one.  The implementation should be updated to take 2 params or the docs updated to specify that the 2nd param is not supported on BB
was this commented by accident?
You have a bunch of these...
queria fazer um refactoring tb, tem uns códigos repetidos.. e fzer um readme daora. Tem um treco de gerar documentação que fica massa, da uma olhada no simpleform:

http://simple-form.plataformatec.com.br/
This was from previous coding... Could you replace it with something like 

```
$this->paginator
    ->getQuery()
    ->setFirstResult($offset)
    ->setMaxResults($itemCountPerPage)
```

(one command per line)
There was no existing eof handling.

It will be handled the same - input will return NULL, so it will return NULL.
why the huge space? :)
Yes, this seems to have broken non-M2 project types' usage of cobertura.  See https://issues.jenkins-ci.org/browse/JENKINS-12640
This is the ``QueryForm`` class, used for search, and search doesn't require a signature to be passed. If you look above this, you'll see that signature is indeed required in ``ReportListForm``.
If performance matters, considering that the regex is pretty large, it's better do something like this:

```python
_ooid_regex = re.compile('^(bp-)?([0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-'
                            '[0-9a-f]{12})$')
def has_ooid(input_str):
     ...
     _ood_regex.findall(input_str)
    ...
```
On prod it is:
"Nightly Crash Trends For Firefox 19.0a1"
Change @wellsize to @padding.

Add a space after the : as well.
Não é interessante usar já o reverse, ao invés da URL estática?
this is going to get removed after a fix today.
I need to use try and catch because I want to see the log message if data is not in history table , I have changed Exception to NoSuchElementException nse. So it is also done
@christianv This one's a mystery. Sublime linter doesn't mark it, and angular doesn't fail (it usually fails on invalid JSON) but jsonlinter.com does mark line 33 and I'm not sure why. Maybe we can look at it together.
I agree - just collapse get_token_and_expiration into self.get(user_id,app_id). We don't need separate methods to get just part of the data.
Yeah, fixing.
There are other ways to do that; this seems designed specifically to show that there is custom css which just can't be edited. I don't really care one way or the other.
Writing this as `domain_enum = 0x80000000` is perfectly acceptable and probably clearer.
Please no general references
You're right, of course--it's a bit of a reflex when seeing "x.astype" nowadays :)
It is accurate but as we are only dealing with binary images it is equal to the area. So we can omit the "intensity weighted".
`(r, c)` -> `(row, col)`
Ah. I don't think I've ever seen the nested `describe` like you show above. I'm fine with having nested blocks, but I think it should be `describe/context`, like in https://github.com/thoughtbot/shoulda-matchers/blob/master/spec/shoulda/matchers/active_model/exception_message_finder_spec.rb#L5
If you have the updated data, why not ``cache.set(...)`` it instead of deleting the key & causing a cache miss on the next request?
https://www.pivotaltracker.com/story/show/25270117
Again, are we sure about this?  Why do we need `equals` at all?  Who needs to know if two polylines are equal?
Is there going to be a new @example? The doc tool will throw an exception if an @example tag is provided without a value as it is here.
They were needed before we were generating a fragment shader with just the right number of textures.  I removed them and everything seems fine.
How about just returning a `Bindable` object? Extending a empty hash with Bindable is equal to creating a Bindable object alone
Please change to KrollDict and remove the warning suppressions and the casting.  Not sure how you tested it yesterday when you got a signature error ... did you change it in Eclipse then not do a full build, and try to test it from inside Eclipse?
Configuration properties are now set via the `.cfg` file.
spaces around commas are inconsistent
Uncomfortable question: Do we need to worry about webviews further down the tree? i.e.

`scrollview.add(Ti.UI.createView().add(Ti.UI.createWebView()));`
Lots of repetitive click handlers here and below -- maybe try a single callback method that takes the id and a filter method to invoke?
Most people write those "how to contribute" docs as github wiki pages (which are not as sexy). It's easy enough to write up the docs using docco, but then it would be a hassle to put them live. 

At this point. I don't see too many people contributing. I'm ankle-deep in building out the lists api, and there's pretty much only sets after that. The contribution docs are at the bottom of my list.
мне кажется, `uploadcare.utils` вполне подходит к "утилитам для аплодкера" %)
Since sorting is handled by elastic search, these options are the same all over addons.  For that reason, they might be better placed within the page object.
Got it.  Thx.
`currIndex` should be declared in the first part of the `find_token` for loops. It's not used outside of them, so declaring it outside of them is leaking a variable. It should be restricted to the smallest scope possible.

I'd also argue that it's better practice to explicitly initialize all of these variables to `0` rather than relying on the default initializer, because then it's clear that you meant to initialize them to `0` rather than forgetting to initialize them, but it's up to you.
I'd say mostly out of habit: The first parameter of the for loop is usually used to initialize `i`, so the initialization of `len = r.length` is almost always outside of the loop.

As far as : 
```D
for ( auto len = cast(typeof(return)) haystack.length ; i < len ; ++i)
```
I find it distils what the loop is actually doing.

I'd say it's a matter of taste, but if you have a good reason for the change (you usually do), I wouldn't mind hearing it.
That check was actually to save some performance by not always clearing the css style if the new value was going to overwrite it anyways. Of course my code was pretty dumb in that regard if any of them were gone it would reset them all. I changed it so that it only resets the keys that are not in the new map.

I didn't know that the watchers didn't have Map support. I can take a look at that.
Do we need the double // here?
Ok, maybe I have wrong information, but I thought that the list of 
service references get updated through the setter.

Am 2012-10-13 06:02, schrieb Andreas Pieber:
>> @@ -110,8 +127,26 @@ public void setEdbService(EngineeringDatabaseService edbService) {
>>       public void setEdbConverter(EDBConverter edbConverter) {
>>           this.edbConverter = edbConverter;
>>       }
>> -
>> -    public void setEnhancer(EngineeringObjectEnhancer enhancer) {
>> -        this.enhancer = enhancer;
>> +
>> +    public void setPreCommitHooks(List<EKBPreCommitHook> preCommitHooks) {
>> +        this.preCommitHooks = preCommitHooks;
>> +    }
>> +
>> +    public List<EKBPreCommitHook> getEKBPreCommitHooks() {
> I though we said to use constructor injection?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/openengsb/openengsb-framework/pull/782/files#r1835477
>
No, this is a slip on my side. Thanks for catching it.

Will leave it on 0, as before. 

Setting it to 1 is a different topic that we could discuss on the list.
you can do this if you force the push to the branch if you want to rebase. But it is not an issue if the pull request contains 2 commits (which is easier: simply push the second commit to the branch and it will appear)
These add overhead, so they shouldn't be included here (i.e. in every page) unless we use them in many places. I'll move them to the obsTableByDate fragment.
I've long since stopped pretending that there are other users of this gem :(
I think this is causing:
      (firstboot:11106): libglade-WARNING **: unknown property `border_width' for class `GtkLabel'

(seems like a strange merge too, but alas, glade...)
Oh, I didn't realize. Is all the code from there?
Why not `using OIIO:ErrorHandler` like the others?
Потому что мне не важно откуда, а важно только куда.
CAS requires that this be https with a valid/trusted cert.
This should be a `{String}`
These are backwards; the expected value is the first argument, the actual the second.
The link does not include everyone that's submitted a pull request. It's a list of users that have had their contributions pulled into the main repository. Subtle but important difference.
запятая после "случае"
Your classing this on root_cron_repaired, but I dont see where the root_cron_repaired class gets defined
Apparently this messes things up on Lion -- if I remember correctly, it causes super to discard all changes made by the current layout manager (self).
Did this so that the DCPU-16 stuff will be separate from any general Assembly syntax, just in case.
It would be nice to have some validation on the table/chain pairing, e.g. "PREROUTING" would not be a valid chain for the "filter" table (is it?)
same as for ReferenceMany. we should make really clear that usually you do not want to specify the targetDocument .
Oh, yeah, just to say, the constants I defined in the CustomerAccess class (and all other data access classes, really) are all Strings.
So it doesn't really make sense to use ```.toString();``` on ```CustomerAccess.FIRST_NAME```.
and that is what is confusing. both solutions involve too much code.
Note: inclusion from "release" CSS file from https://github.com/AllPlayers/allplayers-theme/tree/gh-pages :smile:
remove synapseClient!
类当中的函数就不用加hotquestion前缀来命名了
Might be good to add that it automatically setups up the auth and local endpoints to hit.
Fixed in 0b858f9
Just move the last `)` into the new line, and add `,` at the end.) Maybe you could use keyword based string interpolation? Would it make things more readable?
Comment could be in more lines. :-)
> [..] relevant within the scope of general server administration if this script were used while managing non-vagrant boxes.

Makes sense, however I don't intend for this script to be used for server administration. A full distribution upgrade would also likely involve prompts/dialogs so running non-interactive is going to break the system.
I put this line to "declare" the use of this variable.
Please remove this from the PR.

Btw, you don't need to rename a composer repo to be able to sue your fork: http://getcomposer.org/doc/05-repositories.md#loading-a-package-from-a-vcs-repository
why commented ?

if the permissions are well established, you don't need the before_filter (authenticate_translator)
There's a few issues still to work out with the wording ("pending" doesn't work with our current assumptions about state names, ie "#{status}ed") but the "Begin test..." line and the "#{test_path} #{test_status}ed in %.2f seconds" lines will still be printed for a pending test. They're in TestSuite#run.
Internal to animation only
put reference to Font type definition ?
Avoid including repositories in pom. The dependent artifacts are available in Maven Central
type is the long-ass class name. *sigh* guess I'll be copy/pasting a lot :P
- add admin css from admin function
- remove default tab checkbox
- add table for audio viewer/player
- add reset button + functionality
Should both this line and the one two above use "Microsoft Corporation" instead?
I reworked that section with commit: b265e1c. On the flip side, though, we should make that element look consistent with other 'parameter-providing' elements. Therefore, I have created Jira: https://jira.springsource.org/browse/INT-2463

*Add 'value' attribute to the 'uri-variable' sub-element*
Here there aren't any translation...
Porque, no hay ninguna traduccion aqui?
Are your sure about the boolean priority here ?

A and B or C and D => (((A and B) or C ) and D) ?
`/var/logs/instagram/..` (and for the next line)
This is called from the web platform, this is the callback that put framework uses only so no I don't care for the callback. I don't even have anything to really use the callback for
aha, you can not do it because you are also going through the parties.
u should insert space beetwen elseif and ( : http://cs.sensiolabs.org/
cs (Please us cs.sensiolabs.org) PSR-0, 1 & 2 

```
if (cc) {
}

```