labofsahil/github-event-dataset-2012 huggingface dataset:
Localization and bug fix

== Change log ==
* Add directory i18n
* Move language related code blocks to i18n/en_US/actionable.php
* Modify language related code blocks to support localization
* Create Simplified Chinese translations (i18n/zh_CN/actionable.php)

== Bug fix ==
* Add "Event::run('ushahidi_action.map_main_filters')" to 
  function _map_main_filters() to activate the filter function.
iphone token send

send

`POST /v1/account/iphone_token_set` to the server to update iphone token
Added command-aliases

Added command-aliases to prevent conflicts with other plugins...
Updated gemspec

Hi,

I've made some commits that facilitates for using delete paranoid with recent installments of Ruby and Ruby on Rails.

Regards,
Johannes
The final build

It's like the final countdown... but better.
Timob 11549 Chapter 11-14 unit tests
Wait for adds to finish before resolving command.
Make sure the subject is null if the message doesn't have a subject

r? @asutherland. This is really simple, and not totally necessary, but I wanted to make sure that if we have no subject header at all, we store it as `null`.
Update build name

Change build name to a most meaningful one (it has now the one from navtable). It's just a proposal, I accept other options :)
djangobuilder.py doesn't follow Django's project name restrictions.

This adds a check_projectname() function to make sure PROJECT_NAME follows Django's project name restrictions.  Simple regex to check and it will sys.exit() if PROJECT_NAME is not valid.

Currently, this is allowed:
```
$ python djangobuilder.py --path test-test
```

This commit will now enforce this:
```
$ python djangobuilder.py --path test-test
Error: 'test-test' is not a valid project name. Please use only numbers, letters and underscores.
```

and

```
$ python djangobuilder.py --path 123test
Error: '123test' is not a valid project name. Please make sure the name begins with a letter or underscore.
```
Implementação da classe de Validation

Implementação da classe Validation usando as regras de validation da Form_Validation. 

Alguns métodos precisaram ser re-escritos, pois a Form_Validation só funciona quando os dados são passados via $_POST.

Existe um teste no controller Welcome.
SOS-823 b.privateMessagingContainer is null occurs when a user views a message
new query classes and interfaces
app-hosts-with-weights

Added a new optional variable called app_host_weights in order to weight how much traffic should be proxied to each host.
Model resource full clean

To make it easier to 'clean' a model object provide a new meta option 'full_clean_obj' in ResourceOptions.
The default value for full_clean_obj is False so as to not break any existing code.
If full_clean_obj is True then ModelResource will call full_clean on its object after saving FKs (and many-to-many related objects) just before saving the object.
Test with barclamps, part 4 [1/4]

Testing again.

 smoketest/00-default.test |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)
(#10901) report statuses have displays tipsy tooltip for report statuses
Add --template option to management.build_solr_schema

This patch provides support for specifying a template to use to generate `schema.xml`, and uses the one provided by Haystack as the default.
Master apidoccleanup

Fixes #294; added BB device support to matrix; clarified some text
Fixes #248;
Fixed up the URL to the Push Capture sample app.

Changed the URL for the Push Capture sample app to point to the correct spot now.
Minor updates

* Removed an unnecessary link to a library in the .pro file
* Included author information in the README.md file and updated the formatting
Get rid of dependencies on Qt private headers

As discussed - patch #1: emit => Q_EMIT
Fix Issue #9 - locale.firstDay support.

- [FIX] When no locale object is passed in 
(it's optional), the DRP does not work at all

- [FIX] You modify options.locale instead of 
this.locale

- [FIX] The firstDay property should be part of
the locale object

- [FiX] When firstDay is not 0, even after 
correcting for the above issues, correct 
calendars are no longer produced for
all months/years
Add 'clear' functionality, range highlighting, auto-apply, and event triggers

Add 'Clear' butotn and functionality to calendar
Separate calendar labels from field labels
Add some event triggers (eg. shown, clicked, updated) for users to listen for
Auto-apply/close when right calendar (TO date) is updated
Allow start date AFTER end date to be selected (automatically choose new end date 2 days after new start)
Allow highlighting of selected date ranges
Use a static ConnectionFactory
Created a POM for this project so it can be built by maven

Hi,

I created a pom.xml file so that this library can built by maven and then uploaded to the central maven repository. My version also deletes the libs folder as these jars could be downloaded automatically by maven.

Regards,

Greg
Minor updates to EDD Template: checkout_cart.php

As I was working with customizing/styling the checkout page, I found myself wishing for a few class names to easily target the **Item Name** and **Item Price** table columns. I noticed the **Actions** column had a class applied to it's `td` element, so I thought it would be equally beneficial to provide class names for the other two.

* Added class names to thead and tbody td elements for better CSS references when customizing table.
* Escaped HTML and attributes where necessary
* Applied WordPress coding standards where possible
Fix global name 'bak_file' is not defined error.

Corrects the name of the 'bak_file' variable to 'bak_suffix'.
Global config file

Added /etc/glacier-cmd.conf as additional config file.
Amended order in which config is read so command line overrules local config file, and local config file overrules global config file.
Amended readme.md to reflect this change.
Fixed some minor typos in the readme.

Finally a commit suggesting to change the .glacier config file to .glacier-cmd to reflect change of the name of the tool itself (and to prevent further clashes with boto's tool).
Use overridename if available

The first commit uses overridename as the title if it's available.
This makes it possible to exclude other people's calendars who share the same name as one of your own calendars without having to rename your own calendar (but instead renaming theirs).

The second commit is just a minor speed up as checking a set is constant time (vs checking a list which is linear).
add chinese translations

add chinese translations
zh_CN.mo
Zh_CN.po
Added config/autoload/*local.php to gitignore
Fix inverted logic, creepers DONT damage item entities.
[TIMOB-7712] V8: When backing out of an application a crash occurs during nativeDispose.

See JIRA ticket [7712](https://jira.appcelerator.org/browse/TIMOB-7712) for details.
These are some changes I made before you added me

I moved the MySQL code and added a config.php file. I haven't tested as I don't have the db - could you give it to me (reply on forums)
Updated GPS code
Add correct directory for GApps & Update libcamera

To be able to not have errors during compilation progress, both
Adding the website relaunch post.
Fixes to play nice with updates to Marionette v1.0.0-beta2

This involved:

1) Updating to use updated, bundled version of Marionette
2) Updating underscore.js to v1.4.2
3) Changing CollectionView test to use "onBeforeRender" rather than
"beforeRender"
4) Adding call in ItemView to mixinTemplateHelpers directly
Facebook test fixea
Ogm 131

Drop keyspace is CassandraDatastoreProvider.stop() method when create-drop is used
can't get the parent in a polymorphic_belongs_to 

I can't access the parent object when I use a polymorphic relation between my controllers.

Following the documentation, I have a controller like this:

```ruby
class PhotosController < InheritedResources::Base
  belongs_to :user, :task, :polymorphic => true

  def index
    logger.debug(parent.inspect)
  end
end
```

But parent always returns nil. It doesn't work either with:

```ruby
polymorphic_belongs_to :user, :task 
```
However, it works if I don't use a polymorphic relation:

```ruby
belongs_to :user 
```
or

```ruby
belongs_to :task 
```

Note that my routes and everything else regarding inherited_resources work fine.

I don't know if it is a bug, or if I misinterpreted  the documentation.

thanks for your help

julien
What about non field errors?

Non field errors shown as bootstrap alerts looks much better than default Django <ul>s.
Though this thing needs needs an alert plugin which I think is not included by default.
Core/Opcode: Added few GUILD opcodes

pull them :)
Tela de cadastro avisos semi

Faltando resolver filtros
inline-permissions-with-mutli-groups

please try this and see if it works
Hacking with Gems

What's the worst that could happen if your app has a dependency on a malicious gem? How easy would it be to write a gem that could compromise a box?

Much of the Ruby community blindly trusts our gems. This talk will make you second guess that trust. It will also show you how to vet gems that you do choose to use.
More verbose repository exceptions

Sometimes it would help to have more verbose repository exceptions for general Exceptions like "HTTPErrorException" and "RepositoryException". So I added to output the information of curl_getinfo() to have a better chance to debug things.
LPS-24667
LPS-21997

Fix if statement to allow non-empty subject and non-empty body
bugfix

prevents you from AttributeError: 'NoneType' object has no attribute 'get' (the none object is "attrs")

other possibility:
if attrs is not None:
    self.attrs = attrs.copy()
else:
    self.attrs = {}

seen @ django.forms.widgets.Widget
Changed init script and sysconfig file

Added create L/AS dir if not there.

Changed init script to use variable instead of hardcoded username.
Facebook SDK 3.x Tutorial Update

Update to Parse.framework v1.1.11.
Use new methods available in the Facebook SDK v3.1. 
Will use iOS 6 native Facebook support if available.
Add support for the iPhone 5 screen.
SSL support

I've added SSL support with a new, optional "ssl" option when calling the plugin, which will cause it to use HTTPS URLs into Google rather than HTTP, so it can be used on SSL-secured sites.  Tested and verified working, though I didn't add any tests in.  I also updated the docs minimally and added the build scripts into the repo to make it more completely match the dist.  Thanks for the plugin!
Project burn-up: additional graph to be drawn based on story point branch maximum (instead of leaf stories)

Link to agilefant story
https://soberit.myfant.com/qr.action?q=story:262

Link to WorkItem
https://jazz.net/hub/ccm/web/projects/Agilefant%202012%20SuperAmerica#action=com.ibm.team.workitem.viewWorkItem&id=3606
Added all routes for plugin

The latest redmine no longer provides the default :controller/:action routes, so all routes must be explicitly created for the plugin.
typecast all sleep values to float to avoid error

just fixed a bug with errors with floats.  This seems to fix my issues locally.
INTEXT-30

I have migrated sandbox spring-integration-smpp and change it to use gradle for build. I also added xml schema for the existing channel adapters and gateways based on my understanding how it suppose to work
Weapon Interaction Work
HOCUS POCUS FOCUS

This pull request deserves to be merged. Here's some gifs.

![](http://forgifs.com/gallery/d/188750-1/Dogs_gets_blanket.gif)

![](http://i.imgur.com/xw7TD.gif)

![](http://i.imgur.com/JSgnU.gif)
Fix tracker select on import view
Remove extra slash

There is a an extra slash which prevented the variable interpolation
Fixed Bug in AllDialectsTest:  DbTypes.ColumnTypeMap does not refresh after switching from one dialect to another for second time
Description for Google Play 

Description on BatteryDrain on Google Play was created.
Made it work!

I usend lint to optimize the script a little bit, removed comments from code that was fixed.

The main issue was with remove_element function.
Adding html_safe for avoid safety problems

This patch fixes disappearing problem with rails 3.1.4
Allow for a custom channel file

Allow for users to use a custom channel file that helps address cross domain communication issues.
Convert CSS to Sass/Compass

Hi,

I converted your CSS to [Sass](http://sass-lang.com/)/[Compass](http://compass-style.org/). In my view this has a lot of advantages: it's more compatible, nicer to read, better to extend and it allows us to drop the `.btn-auth` class requirement because of Compass' awesome `@extend` statement.

To compile the `.scss` file to `.css`:

     $ gem install compass
     $ sass --style expanded --compass auth-buttons.scss:auth-buttons.css
.rst cleanup

some cleanup including running point totals and updating the total possible points.
us variant battery mA fix

battery mA fix
обновил версию плагина, чтобы нормально вставал в OpenX
NXCM-3654 - Changed updateSiteMirrorTask to restore groups support

Since core merge repository and group at same property this task was failing to run for groups.  Now it works just fine.
Compatability with new JMS\PropertyMetadata interface

JMS\SerializerBundle\Metadata\PropertyMetadata interface for $type has changed.

Here is a quickfix for it.
Do not commit minified files to git

Source files should be stored un-minified in source control and minified using a build tool like Java#CoffeeAidScriptified Singletonion Mochasyntactic Idiomationed Nommifier BangPlus (JSMIN!+;)
Improved functionality and added unit tests

Hi

I've implemented the possibility to copy content from more than one source directory into build jobs workspace. Additionally the check of the given directories is improved by a check for their real existence and some unit tests are there too.

The plugin with these improvements works as a charm here on my Jenkins installations, so please review my changes and pull them into the official plugin.

Thx a lot,
Yves
This should have been merged a few days ago
Cm 2.6.35
added a few Techly goals

Added points for:
Facebook hook in,
CoffeeScript
python
OpenShift or other PaaS
build sensors and lights libs from source

The sensors are the ones used from the exynos4 hal libraries. they work fine in my testing. as for the lights source, it's an import from the crespo4g with some modifications to work well with our capacitive buttons and lcd backlight.
exercise2: I am a horse in the land of booleans
Use documented means of adding matchers

Extending jasmine.Matchers.prototype directly relies on
undocumented Jasmine internals. In particular, it won't work
if the jasmine.Env is created before the underscore matchers
are loaded, as is the case under Jasmine's Ruby test harness.
Fix for battery styles not showing up when Greek language is selected.

This or completely remove the translatable.
Improved BootReceiver

Removed the need for StringTokenizer and made it more compatible if ro.pa.version formatting is changed. "else" statement needs to allow it to work at the right times.
Add show and list "time" templates

Related to PR #475
Update Twitter Bootstrap to 2.2.1

Update Twitter Bootstrap to 2.2.1
General Fixes

Went through and cleaned up all warnings and notices. Also added the language files provided by @thomasclausen https://github.com/sproutventure/search-everything-wordpress-plugin/issues/24
Adressed issue #2
Array: Max Array Value – Use are more elegant way to determine max value in ES3

See also https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/Math/max#Example:_Using_Math.max
@ericf I completely agree.  The only reason that behavior would exist would be to preserve workarounds current in production.  I'd be perfectly happy just changing the subattribute + getter behavior in a backward breaking way.
Should the plugin require `cssbutton`?
Double negative... undefined means that the state doesn't change or undefined means that the state does change?
Now I at least know why I only saw it on my windows dev laptop ;) Anyway now its fixed..
Fetch leaves the k/v in the Hash, probably don't want that here. :(
This is a nice event to have, but since it's not currently being used anywhere it's probably prudent to remove it if there's no easy way to catch it with OIS (in SDL it was just one of the events one got when polling).
No worries - thanks @cswegger
You don't end lines with semicolons in Python, they're for separating two different expressions on the same line, like so:

```python
if x is None; print "spam"
```
Обязательно ли тогда надо поддерживать tuples?
The `id` attributes are not needed here.
Mmm, for me File.directory?(path) is more to test if an existing path is a file or a directory, not really to test its existence (even if it works for that too).
好吧……刚才发现丁香园的serial number是个大数……我放弃……我想知道别的浏览器遇到啥问题了……
Miss inline qualifier
After your second commit I retried, but my suggestion failed. It seems that it's some bug in ascii/pandoc.

The table is exported to pandoc fine, but there is some problem about the number of spaces before the first title in header. If I add there some extra spaces, then there is no problem... Strange.

 Will investigate, till then you could:

* add a block like the following between the list and the table (will not output anything, no worries):

```
<%
'<!--- end of the list, ha! -->'
%>
```

* Or just add some sentences between the two :)
ping result no more depend on text, but return value, $?
How does this handle multiple zfs filesystems?  It feels like there should be a "zfs_version_<name>" type fact for each zpool or dataset (I don't have a system to hand to check the granularity).
DOCS-157
why is the same parameter used twice here?
ah, sure. thanks for pointing that out
nah, i'll just make a new issue ;)
Shouldn't min/max-width be calculated earlier, so it can affect other dimensions?
Check out my python-organization branch. That should fix the issue of not being able to run in place :D
just responsys?
I'm guessing this works too, but I'm curious: any reason we chose not to call set(0) instead?
average --> median
Yeah, we can clear the keys in env if they aren't in saved_env.
this is a bad system. the packet that it sends is bad. For example go on a druid and cast feral charge see what happens. I checked this method out when it was commited on skyfire.

The problem is the spells are supposed to be learned temp. But you wont see " learned spell #" You don't want that spam that looks so bad.

Either way check the packet its bad.
Mmm, red is such a pretty color in diffs
This can be improved in a few ways:

1. In order to avoid depending on zope.app.component the getSite import should be:

        try:
            from zope.component.hooks import getSite
        except ImportError:
            from zope.app.component.hooks import getSite

2. getSite returns the active component registry, not the Plone site. (It's different for KSS requests and in cases where a developer has set up a section-specific component registry.) Better use something like getToolByName(getSite(), 'portal_url').getPortalObject() or go up the acquisition chain of REQUEST['PUBLISHED'] until you find an IPloneSiteRoot or something like that.
Nice catch! Just pushed a fix....
Make sure you always indent with 4 spaces. Indenting with 2 (unless all the code is 2) is frowned upon.
Oh I know what the problem is: `div(Unsigned,Signed)` changes the type of `x` in `x=div(x,10)`.
Sry...hast's ja wieder rausgenommen ;)
Because I couldn't get the python parts to compile correctly, didn't need them, and didn't care enough about python to fix it.  I figured if someone who wanted the python parts came along they could build a libnewt-python package for it..

On Feb 9, 2012, at 3:47 PM, Dave Tapley wrote:

> Why do you remove the installation of python bindings here? (I'm assuming that's what's happening)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jasonk/homebrew/commit/29a263bda3ec5acecbb899e581e7ba953ae12c85#commitcomment-951125
>
额，那就用Doxygen吧
обновился сёдня с этим коммитом 
http://www.youtube.com/watch?v=SMBFPz2_rlw&feature=player_embedded
/root/mangos/src/game/Unit.cpp: En la función miembro 'uint32 Unit::DealDamage(Unit*, DamageInfo*, bool)':
/root/mangos/src/game/Unit.cpp:868:84: error: 'damage' no se declaró en este ámbito
make[2]: *** [src/game/CMakeFiles/game.dir/Unit.cpp.o] Error 1
make[1]: *** [src/game/CMakeFiles/game.dir/all] Error 2
@peller why did this need to change? leaving off the 'this' third argument to dojo.forEach in the new code broke it.
run check.py
а вот если использовать formtastic было  бы строчек 8
this is breaking [ci](http://travis-ci.org/#!/radiant/radiant/jobs/607842). @jfahrenkrug could you take a look?
this is 90 chars long. 79 is max.
I think this is good solution If there is necessary  start vm with specific number of cpus.
Note that thismonth and thisyear still need to be rendered to this page
And 'thismonth' is probably not a good name, since it is already taken by the object that contains the month name, the days and weeks of the current month page.
![example](http://i.imgur.com/25J4h.png)
Good call. fixing
we can of course just import it directly, like ``storage = WatermarkStorage()``
если считать что этот UserBannedException мы выбрасываем только в whois (а это, наверное, не правильно - надо бы еще и в LoginController делать), то можно custom'ный обработчик исключений в контроллере сделать. Там какая-то аннотация для этого есть. А уж в обработчике можно сделать setStatus и вернуть jsp view
Ahah :)

http://www.linuxjournal.com/magazine/pointcounterpoint-opt-vs-usrlocal

Le 03/01/2012 10:42, Nicolas DUPEUX a écrit :
> Probably came from my installation of shinken. /usr/local sucks ...
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/naparuba/shinken/commit/dd084b8c5ad25a0786fbe79733345d14cffbae2a#commitcomment-833063
Hm, http://developer.gnome.org/gtksourceview/stable/lang-reference.html mentions a gtksourceview-2.0 directory.
Should keys be `[:one]` here?
Fix the issue-175: assertEquals(Set, Set, String) is missing
Otra vez el git add . =)
やっぱり実行部分のコードが分かりません。
つまり、ロジック部分も理解できていません。あわわわわ。
Finally I had time/remembered to merge this (https://github.com/mpeltonen/sbt-idea/issues/62).

Thanks!
oh duh it's a symbol. heh. symbols shouldn't have capital letters. >.<
how does pulling down a cached version of a jquery library from your local cdn equate to "being tracked by google"?
Rollback! After update stream empty and top header disappear
Excelent! Please do so :D thx!
Love how nice these banners are. :D
I wonder what's the probability of that happening. But you're right. 

Anyway, the IDs stay, so there is a way of unambiguous identification of components. If we don't suggest that the pointers are immutable, nobody should assume that.
If you could, could you look over this? I'm not sure what the best way to approach this is, since it returns an array of length 3, and we are only concerned with indeces 1 and 2. 0 is just useless crap, usually a blank string.

Right now i have it working by using a throw-away variable of `cruft`, but that isn't ideal. Perhaps there is some syntax i, as a new ruby programmer, am unfamiliar with
That looks like trailing greenspace to me.
ah, nice! I assumed unicode would have just used code point characters
Why not call system.exit(), as it s equivalent? Is this being called in some sort of exception trap?
Oh yeah, the frequency of logins might do it... I'll think about some ways to approach that.

After it's threaded, won't the latency only be an issue for the person connecting?

Given that it's not yet, isn't that a vector for DDoS attacks?
void SpellHitTarget(Unit *pTarget, const SpellEntry *spell)

Well I Think this is incorrect....
yes. The version of getTileAt I posted above checks pointers validity and you need to change the current one in DetourNavMesh.cpp with the one I provided. You will have to add the includes I posted in order to compile.
will fix (and rename to addParenIndent)
You already tested s is not empty on line 77.
oops sorry, didn't see b089e48b8934c52741e1ec7fb369ddd202da90a6
Jamie,
keeping it generic is useful for local development. Setting up Postgres is a pain.

Boris

On 17/01/2012, at 22:55, xrymbos<reply@reply.github.com> wrote:

> We've already moved to postgres, SQLite was slow as balls. Is it worth the effort keeping this general in case we want to move away from postgres?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/xrymbos/nztrain-v2/commit/fde356ae57afdb6b8e746bd5a95bfaec04b2f224#commitcomment-873649
1. 估计Zj = 张角
2. 考虑下司马懿的鬼才
3. 名字真的得名字改下
@neilmiddleton yeah, I expected it to pass, I just wanted a regression test added.
@joshk, strangely [the build](http://travis-ci.org/#!/travis-ci/travis-ci/builds/730598) failed on travis-ci and I don't know why. Specs run fine localy on my machine.
May you can put it in a pull request for tc ?
эта строчка повторяется 3 раза, может ее можно перенести в ApplicationController (если позволяет логика)
(copy-keymap popup-menu-keymap)では動かないでしょうか？
Je débute avec Javassist, donc je ne sais pas s'il existe un moyen simple de faire la réflection à partir de la Class java du test observé. C'est notamment pour ça que je suis intéressé par un coup d'oeil tiers :o)

+1 sur l'injection. Le runtimeEnvironment me donne accès au classpath d'exécution du test (et donc à la classe testée, ce que je n'ai pas dans le SlowTestMarkerInfo.
E com relação aos demais métodos do FacesContext, fariam parte também do utilitário Faces? Lembrando que ele não foi concebido como Wrapper, mas se o considerarmos como tal, seria coerente que os demais métodos fizessem parte dele também. Será que vale à pena? Acho que vai ficar confuso: ou é util ou é wrapper.
貌似是这样。。准备改的 不过乃已经改完了
I just added the mechanic that were left, how do you suggest add immunities to bladestorm?... if you see down this commit, there is a mechanicimmunitymask that does the exact same thing i did here with other spells like hunter's beast within.
Why the old event system?
Ah, it think it’s going in the direction opposite of your reading – iterating through all the keys and checking if they are defined on the object.
If you could push this new version to rubygems, it'd be fantastic. Thanks!
they are looking for any date...  :)

It was hardcode to be a particular time zone.  The fix just moves all
updates to be UTC.  It just seemed the best route, standardize on output so
we don't have to worry about -700 vs. -900 stuff...

Greg

On Mon, Feb 27, 2012 at 10:55 AM, gcoco <
reply@reply.github.com
> wrote:

> I assume that Strava is looking for a file with UTC time. Is this correct?
> If so then this looks fine to me.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/glonnon/GoldenCheetah/commit/c5d0dcbf1c60ca2087e8fa8cf2aa652a6e353e47#commitcomment-1016211
>
well both work for me and they should..
This change made XCode 4.2 unhappy (complaining in the NSError+Git git_errorFor:withAdditionalDescription: function that it couldn't find gitLastErrorDescriptionWithCode:.

I moved the implementation of getLastErrorDescriptionWithCode to be the first function in the NSError+Git.m file and that made my compiler happy. Not sure what you *really* want to do here, however.
defines the default constructor for the class
Isn't priority 10 default for add_action?
.Net naming: Maybe use `Icq` instead of `ICQ` (As Xml, Id, etc)
Can you do these as:

       "fileupload": "~0.0"

from now on
so that's actually a next step for me--I'm going to extract the messaging logic into a behavior and then be able to use that to create the first meaningful eunit tests
The save operation should only know about adding insert actions to the ActionQueue. Logic for executing actions before a save has been moved to ActionQueue.addResolvedEntityInsertAction().
typo: amound
You also need to change it in /includes/core/config.php ;)
Yeah I did that first up but the new duplicator removes all the instances from the DOM and adds back when needed.
That's actually not heading to anything, it's just playing around with options and getting different (bad) results

Only way to fix it is to debug your code flow and figure out why clock values end up to such abnormal values.
interesting. how much time is usually spend locating start/end points, and how much in routing?
earlier i was having issues with shared credits not being given, 
but frackit lol everyone still gets a +1 for getting this working, 
something more then had did b4 fo'sur. 

which is more of the point "that it had gotten fixed" 

its all good man, 
i just got pissy, earlier is all. that moments passed, 
and back on to more developments as always xD
This used to be testing limitStepBlock - was the change intentional ?
Looks really good. But please try and work on the actual jade files. I know it's harder, but it causes a lot of problems for me to try and move this stuff into the actual project.
Sure, that's fine, but one pull request/issue per feature/fix, please ;-)
Oh you mean included in the core.. nevermind :) Risky or no? 
That will be next if the plugin doesn't work
@will-moore: I will not have time to do it next week, I will do that next sprint.
I haven't know about such way. I was using 'writer' at first time.
@stephenkostas - there was only one comment that was not addressed...

Installation Requirements should state the following for Java JRE:
  Java 5.0 JRE (or greater)

Can you update that last thing? Then it'll be good to merge into master. Thanks!
This docstring should explain what the method is used for, not what the code does -- that's pretty obvious.

Actually this works without a docstring -- just rename `value` to `decimal` and it gets pretty clear what this method is about.
I added true/false by accident because I thought about reusing this "parse and read json file" for -kr. I failed - would have been to complex. Then I forgott to revert my changes. I don't think we have to make a final decision about this yet.
OSS rox ! ;)
Wouldn't you want && not ||?
Use `HttpStatus.SC_PRECONDITION_FAILED`.
Example of required documentation: who does the byte* belong to? Who should deallocate it and when? Can the function fail? If so, what happens?
Me too, but I'd rather not have a method call overhead on every record on a first sync. The null check in the filter invocation location will be JITted away.
Any ideas for nicely abstracting the 2 lines before and after the query in both safeQuery() methods?
Ah, nope. I'm wrong.

It was applied. Only the version number is missing, all other changes are in there.
Nothing to worry about. :-)
HI, Can you give me a short explain on how did you change from LGPL to GPL ?

Thank you!
But I like the approach. Shall pull later on...
What API would you want, ideally? We can shift some stuff into a method in core if you like, that you can override here…
> Is that a typo?

No, just poor English from my side ;-) I'll change that.
My understanding is that the options are cached by WordPress, and get_option will always attempt to use the cached version first.  So the code in this pull request would be just as performant as what is being currently used.

When I look at the page load in Debug Bar (http://wordpress.org/extend/plugins/debug-bar/) it also appears that all the requests are cached.

If I'm incorrect, please argue me down and help me out with a couple examples.
What nimrod added this?!
From what I've retested it doesn't break anything in the major frameworks so I'm fine ether way :)
Lol ..
Indeed.  Not sure how I bungled it, but fixed in 282ee0fb20580a742e4a22b890b2a352188c8bf7.  Thanks for reviewing.
@rikarends ?

On Thu, Jul 5, 2012 at 11:09 AM, Zef Hemel <
reply@reply.github.com
> wrote:

> > @@ -480,7 +482,7 @@ function startMonitor(tag, flags, script, args) {
> >                  stopProcess('SIGTERM', true);
> >                  break;
> >              case "testexception":
>
> What's this?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/c9/runjs/pull/8/files#r1107186
>
What's the TODO here?
Public to allow features to be added to Promise.prototype rather than needing to subclass both Y.Deferred and Y.Promise to add features. Made for less, and less complex code. *shrug*
this should be Jasig
Why did you remove this section? We still have the concept of a singleton cluster.
Should the log be more semantic than technical? 

"Cluster Node [{}] - No cluster convergence, since not all nodes have seen the same state yet [{}]."
I think the appropriate thing would be to disallow Predef.Seq :-)
No kidding.  At least my 11 years at Sun were mostly on the infrastructure side :-)
This might want to be a `reduceRight` instead, since that would allow short-circuiting (not a huge effect for sure).
dropped the executor?
This is a bit half-baked, isn’t it? Either do it fully with nice indentation or leave it on one line (in which case you might just not override at all).
what if multiple threads do start at the same time?
do we need to support that?
Good question...  The short and probably incorrect answer is that they could use scalaj-collection, create a list and use .asScala.  I made this change based on an earlier comment.  Originally this was an Array[Address], but the mutability of an Array was distasteful.  Can you suggest a happy middle ground?
this does not actually use `ec`, should probably add some combinator to the example
I pretty much eliminated references to User in favor of delegated references through Account, and I added a couple of those.
ok, distance VS threshold... Dunno then. distance_threshold ? or drag_threshold is ok.
char.IsDigit is probably enough
No need to inline inner exception message, it's already included
And ``HemiSphere(sphere)``.
I put it back!
For PEP-8, no blank line here
An oversight.  I'll correct.
I believe this one is worth going through the scenarios for.  Probably just clearing it with all the set stuff is correct as that has to clear almost everything else (I think the method is reset_patterns, we may want to rethink resetting/clearing more deeply at some point really soon as well).

So basically I agree and am arguing we want it here.
I will add an "if UsesEncoding(file1)" (and obviously also for the one for file2) here as well. There reason I did this is because the outcome would be same as with the removed ::Decode() below
ok, a bit of djangoness is never a bad idea... ;-)
remove this once you can set nils
dc = pa.status_stamps
This has nothing to do with Behat. It is the way Composer install things. And look again. What you have in bin is a symlink to this file, not the file itself
@dizzyd Regrading `compile`, I just followed the original behavior where it was calling `check-deps`. `check-deps` was changed to call `save_deps_dirs/2`, because, unlike `compile`, it should work recursively on `raw` deps as well. The full list of deps will be returned later by `postprocess/2` and `raw` deps will be processed recursively from there. The same applies to `get-deps`. I relied on the existing mechanism in `rebar_core.erl` to avoid (post-)processing non-raw deps twice.
not sure if bounds need to be checked here because these are used to determine the sign of multiplication
Should I add it to TransitionScrollStrategy.js too?
Having days can be nice, can't it?
I understand the `_` part, but why make it a property? Apple's definition of property pretty much does not apply to this limited-use variable.
I'm of the opinion that associating these with a content view _controller_ is infinitely better than just a view. This doesn't seem to make much sense.
Please change this to return early on failure, and pull out this ternary expression.
Why does this need to be protected against being invoked multiple times?
Should be: integer
Looks good to me, and yes, I definitely want to still use q.
`AbstractOioMessageUnsafe`
Oops!  This file was not meant to be committed. I've kept it around as a 'just-in-case'.  I'll remove it from the pull request.
не выйдет
'available_environments' method is a private class method. It works with file system, so I stub it in the test.

Also, can anyone point me to a guide on rails tests?
non-breaking space:

```
< > 160, Hex 00a0, Octal 240
```
the slow but correct  way would be to call typedIdent on a typer in that context, i guess
No, will fix.
file:///pack/lib/path#http://bog.us
Can we not make it even harder than it already is to build with java7?

    init:
         [echo]        build time: 11 June 2012, 14:54:04
         [echo]      java version: Java HotSpot(TM) 64-Bit Server VM 1.7.0_04 (1.7)
         [echo]         java args: -Xmx3g -Xms3g -XX:+TieredCompilation -XX:ReservedCodeCacheSize=256m -XX:MaxPermSize=384m -XX:+UseNUMA -XX:+UseParallelGC 
         [echo]        javac args: 
         [echo]       scalac args:   
         [echo] scalac quick args:    
         [echo]          git date: 20120611-145108
         [echo]          git hash: 5773123b54
         [echo]     maven version: 2.10.0-SNAPSHOT
         [echo]      OSGi version: 2.10.0.v20120611-145108-5773123b54
         [echo] canonical version: 2.10.0-20120611-145108-5773123b54

    BUILD FAILED
    /scratch/trunk6/build.xml:337: This build requires JDK 1.6

    Total time: 2 seconds
guess it's better to remove py30, pypy and jython -- scipy doesn't work with those.
Dangit, sorry, I had imported the release version, not the dev version. You're right, this can go.
Oh, I added that too. I just thought the docs should reflect this reality as well.
You might want to check that the length of the args is 1 before you do this and raise an error otherwise.
By the way, the convention is to separate function and class definitions with one line (at least that's what I've seen everywhere throughout SymPy).
Showing `(g * f).components` now.
I didnt know this fact. Why not just call it is_alternating and note that it can be combined with is_alt_sym to determine if it is symmetric too?
Add spaces around `-`. Remove one extra newline.
Well feel free to open an issue.
oly->only
OK, I can add check that if you do it, then exception raised. Or I can just document it.
I realize it was already this way, but obj->GetKeyName() is less characters than obj->GetKey()->GetName()
That makes total sense. Fixed in 7b32008
people willing to create another mailer should not name it `mailer`.
Вот этот импорт нужно убрать, чтобы тесты не падали.
You need a mutex here. Multiple threads will access this value concurrently.
!!!!

 :hosts => new_resource.hosts

!!!
=> " , "  ;)
I just noticed that this is passing in the map and expecting it to get mutated.  What is the benefit to forcing the strategy to mutate a map rather than having it say, "here's your node"?

It looks like you are using the Maps as shared state between the strategy and the TaskRunner, but sharing that state is blurring the boundary between the strategy and the TaskRunner.  It would be better if the strategy were able to encapsulate all of the state, or if the TaskRunner maintained all of the state and the Strategy just provided methods to tell the TaskRunner what to do and how to do it.  I'm generally more in favor of the latter.
isn't it "tracks" because the subject is "all mongod and mongos instances" rather than "each mongod/mongos instance"?
Are we shooting for brief, or complete?  You can specify the service name, service "display name" and service description if you feel like it.  Not worth mentioning if "brief" is the goal.
Hmm, though this is good for the sake of completion, we probably don't need to undo setting is_redirect here. If this migration is undone, it's likely that we're next going to drop the field - in which case, this is just extra work to wait to finish.

In fact, this migration should probably be merged into 0021, so that adding the field and populating the field are done in a relatively atomic step.
`if (!empty($queries))`?
Has been added by d5cbeda
Check whether there is a function for doing this. Consider adding one if none exists.
I've seen file_exists used elsewhere in the codebase for this but I agree it makes more sense semantically to use is_dir so I'll do that.
Please audit the rest of the diff for missing spaces after flow control keywords and comment symbols, and your other diffs also.
An ID is needed for the onclick event that appends #preview to the action of the form submission. I'll change the ID to something like qrpostform instead.
nit: that is a weird comment style
I guess there is no need for the message, since the activity has `public=False`
I'm trying to understand your reservations about the delegator here, but not seeing it.

My goals were:

* Provide some logging mechanisms totally decoupled from anything else in the app
* Allow other code (including code outside of the `shoes` gem) to register and use new loggers

The delegator is simply there to provide setup (e.g., requiring java) before delegating things like `#debug`, `#info`, etc.. to the "real" instance.
It's not a "completion callback" or "error callback" but it *is* a callback.  walkSettings() calls back into the other scope for each file.  It fits the definition of "callback" in my mind.
Удаленный баннер != баннер с истекшим количеством показов
Ignore statements like these should not be added to this (or any other) file.
This is so, so awesome. And so Julian. People are going to love it, and it will feel totally different from Pandas or R. Thank you for working on this!
could use foreach instead (aka "for .. in")
http://www.w3schools.com/js/js_loop_for_in.asp
style... we are using camelCasing throughout. Doesn't need to be fixed immediately, but FYI for future commits. We'll have a code-cleanup phase in 2-4 weeks in which we'll address style issues / coding conventions.
* Is this going to prevent others from launching all the modules?
* Also, shouldn't have commented code
Sorry already merged. Would you mind re-basing the load api PR and handling this conflict (and I will then merge).
Can you add in the docstring what the input parameter are and what the output is?
I read articles about Fear Movement in Arenas/Battlegrounds being disabled during Wrath. It was returned back in cata, but i can't believe this is sure. I'll change this back if necessary.
"http://" used at line 89 & 91 can be replaced by the global constant AT_SERVER_PROTOCOL.

@ see https://github.com/atutor/ATutor/blob/master/include/lib/constants.inc.php#L183-191
The problem is solved ... !
Oh I remember why I didn't do that now. It's because you can create new types, and for instance c_sint_type and c_int_type are the same, but don't compare equal (I'm not sure why?).
No reason, will fix...
Missing space after IF
by just making the warning fatal, you never verify that the behavior is actually correct.  that's why i did the showwarning nonsense.
Nitpick: The default for a field is `NULL`, unless otherwise specified. The style we've used so far is to simply omit the `NULL` keyword. (See `number_of_logic_tree_samples INTEGER`, for example.)
Why is this designed to split the calculation in a fixed total number of tasks, rather than a concurrency level (like in `[hazard] concurrent_tasks`)?
by node itself - Emitted when the process is about to exit.
http://nodejs.org/docs/latest/api/process.html#process_event_exit
HTML5Lib returns dicts with tags split into their contents. This means f.e. that the following html:

```html

<p>This is a text.</p>
```

might be returned as

```python
{'name': 'P', 'data': 'This is a text', 'type': …}
```

So what happens in the whole block (I'm starting with l80) is that I'm trying to be "smart" about the tag. Is there another ``<script>`` tag inside the ``<script>`` tag? If not then I'm moving on to detect what is happening inside the ``<script>`` tag. It might be possible that someone is just passing in faulty html, which might mean that there is a script tag without a closing end. However on second thought I'm not sure if this is needed in any way and we should just skip the token in all it's glory. Need to test this.

The list is not complete because I was trying to reduce the keywords to the ones which are *common* javascript. I couldn't think of any line of javascript which would not have at least *one* of these strings in it. However, as I commented it in the line above the check, I'm not quite sure if this isn't too dumb.  I'd be happy to see me proven wrong by some tests!

When it comes down to performance.. Yes this is run on every token after an opening script tag until htm5llib tells me it's over. I didn't do any benchmarks on this. It might be a bottleneck on *huge* script tags, but usually python's iteration and ``string in foobar`` are pretty fast. One thing which might be pretty would be to move this on the top so the tuple get's loaded on import.
Anyways, a regular expression might be faster.
Really?  Hmm, I'll test it later...
A checking of result.isNull() is probably needed.
Move the comments which is applicable for general parts of RF to somewhere common.  Duplicated same comments here and there is not good.
Right, and I've never seen such environment where perl is not available except Windows.
Unfortunately I have to deal with both Android and other platforms.  I just went with the store/restore method which works in all cases even if it can generate a little more wasted state changes.

Still....  does GL.Finish() need to go back in here or not?
Should we protect against both being set?
Use

```rb
system "#{bin}/localinfo"
system "#{bin}/sip-date"
```
> sorry I just realised my previous comment was completely lacking social
> skills.

don't worry. I'm pretty bad at social skills on github usually :)

> I installed pySurfer this morning using easy_install and the nested try
> except that you just mentioned being in the master was not there yet.

ok we need to make an new release ...

> but yes, this seems good. I changed my local copy of viz.py.

or use the latest git master

> Thanks for making pySurfer by the way. I like it.

excellent !
Do we perhaps want to elaborate why we're not using this?
just remove the line, it's going to be in the history
Why are we using an additional argsLength variable ?. The arguments.length is only evaluated once.
Could you please remove the spaces between `=` here? :)
I would.  Just now, no, because we control what two plugins are given and in which order.  But if we do open it up to allow someone to provide an alternative plugin, we simply have to assume badly written plugins, and fail accordingly.

So... maybe overkill just now, but a wee little bit of experience with crappily written eclipse plugins'll teach you - always make sure your plugin apis are guarded where used.
(Also, none of the tests failed :D)
Can you split this into two tests?  Thanks.
should probably use getOrSet
Does this actually need to be a subclass? It doesn't seem to be required that it's a separate class. What do we gain?
This can be reduced to the widget that is in focus since the other widgets shouldn't handle keyboard presses when they're not focussed.
Clean up these comments
Actually, I don't know why the left-hand side of that `||` is even needed. You're not allowed to pass null to `setCurrentDocument()`, and if you do it'll just crash on the very next line anyway.  Can you just trim that part off?

(I also think the `||` probably should actually have been `&&` (without the `!`) instead, but that's moot if we're removing it).
I actually think this would make sense as a public function--it makes sense that a general purpose color widget would provide an externally visible way to set the current color in any format (as opposed to just having `setColorAsHsv()` be public). If we do make it public, it would make sense to just change its name to `setColor()`. (And, in fact, you could argue that `setColorAsHsv()` should really be private, since it's really just internal glue code between the UI and the model--the fact that it's set up to only modify parameters that have been changed in the UI reflects that.)
These two lines caused the Brackets to always connect to Chrome, no matter what, which is not as intended.
please change it to
```javascript
if (!name)
    return "";
if (name.indexOf(...
```