labofsahil/github-event-dataset-2012 huggingface dataset:
Add collective.solr to github collective

See: http://lists.plone.org/pipermail/plone-product-developers/2012-August/011747.html
Adjusted readme.md

* Description:

* Notes:

* Actions:
Change wording for editing routes.

I've seen someone append to the existing line, so I'm trying to make it more clear. Is this any better?
OAEWDGT-98 - Inconsistent indication of optional fields on Submit a widget page

Adding better indication of optional fields on submit widget form
https://jira.sakaiproject.org/browse/OAEWDGT-98
Add missing calls to configureShowfields on AdminExtensions

Resubmitting against correct branch :)
wrong assignment of nextText
Added example on how to create radio buttons dynamically.

Radio buttons can be created dynamically using the value binding.
Invert option for autoscroll

Hi,

now for dev branche.

I Added a invert option to autoscroll plugin. If set true the items will scroll backwards (api.prev instead of api.next)
Hope this is the right way to contribute :)

Thanks for feedback & Regards,

Slavic
[CORE] Add OnAuraDispel hook on AuraScript

This hook is called before and after the aura is dispelled and gives the dispel informations (dispeller, removed charges or stack and dispeller spell id). It's usefull for many spells.
Core/Spells: Update nearby visible objects for new viewpoints

This fix spells with Far Sight and Bind Sight effects.

When the viewpoint position is different than the current player position, nearby objects should be visible. 

Spyglass items aren't working yet, and sentry totem have some issues.

Spells now work correctly:
Eyes of the Beast (hunter): http://www.wowwiki.com/Eyes_of_the_Beast
Eagle Eye (hunter): http://www.wowhead.com/spell=6197
Eye of Kilrogg (warlock): http://www.wowhead.com/spell=126
Far Sight (shaman): http://www.wowhead.com/spell=6196
Steam Tonk Controller (engi item) http://www.wowhead.com/item=22728
And other non-class specific spells.
updated namespace list
Added javadoc target

I think the javadoc target is to be included
Add Ubuntu support to module.

- Add case statement to set packages and buildoptions based on OS and OS Release.
- Change buildoptions class attribute to avoid conflict with buildoptions set in the case statement.

Needs tested with Debian Squeeze. I tried but couldn't get standalone puppet working properly on a debian vm. I would gladly test it for you if you have a virtualbox vm or vagrant basebox that I could test it on.
pom.xml for distribution

I've changed pom.xml provided by @bartoszgadzala according to Sonatype requirements (https://issues.sonatype.org/browse/OSSRH-3390) and added distributionManagement section
so maven artificats would be signed with your gpg key and deployed automatically by using:
```
mvn release:prepare && mvn release:perform
```

rem: KFramework-SW usages were commented for now; we need maven artifact for it, too
add modem and q6 firmware files

ville: HTC Europe 2.31.405
jewel: sprint 2.13.651
evita: ATT 2.20.502
jquery tab for options not working causing options to remain hidden and ...

...inaccessible.

Removed tabbing class, so options are always visible in doc.
Remove extraneous directories.
this add class for ParamConverter without class

Hi,

This add the possibility to configure the paramConverter without class param, when the parameter has a typehint:

```php
<?php
use Acme\BookBundle\Model\Book;
    /**
     *  @Route("/edit-book/{id}", name="edit_book")
     *  @ParamConverter("book", options={"option1"="foo"})
     *  @Template()
     */
    public function editBookAction(Book $book)
    {
    }
```
Surround @Cache annotation in quotes and fix comment typo

Currently the unquoted @Cache string in CacheListener is causing an error. This PR fixes the issue.
problems with SRIV and NAs related to filter() 

Hi Joseph

Didn't realise you've been so active in pushing up new features. We should probably move to your branch being the canonical source.

In the meantime here is a fix you might want to merge in. (keep your own version numbering in DESCRIPTION, obviously)

Would be great if you could build the windows binary package too.

Cheers
read files in binary mode to avoid windows file reading issue

fixes issue #189
Avoid network I/O for remote storages in get_filename

I haven't run the tests on this because I'm not sure how to at a glance. We're running an equivalent monkey patch in production that avoids the occasional timeouts we were getting when `exists` was called for every file to S3.
Templatetag should fail silently, rendering original context.

I believe this is beneficial over raising an error for a use case where you have a multi-server deployment and a single shared media server (This may not be the most common setup, but I believe it is common overall.)

In a deploy, the manifest.json will become out of sync with compress blocks (if modified) and while a deploy is happening, the hash keys will not match up. 

You may feel this is out of the scope of django_compressor, as we should simply handle our deploy differently (e.g. taking servers out of rotation) but also feel this would save some headache for many people with a similar setup.

I would be a fan of importing the logging library and sending a warning in the cause of key failure. This adds no dependencies of course, as it'd use the built-in python logger.
Add array/combine() and array/append().
auto_id = False for multiple BrowserID forms per page
Daniel/bii

der basiert auf deinen bii branch.
Respect "noWrapper" setting

If "noWrapper" is false, leave off the '-b' argument when compiling.
SVG Support

Hello Ben:

I made some changes to have all elements check for an ownerDocument before proceeding with calls to getComputedStyle. The new method is called "_isValidElement". This method is called before proceeding with _interpretValue and _appropriateProperty.

I hope that this is the correct approach for generic SVG support.

Regards,
Glenn
added tool bar etc
Added attributes for orderby and order

Just added attributes for orderby and order so I could sort by menu_order.  Updated version number.
Travis CI integration for Grunt QUnit test suite

This code adds support to run the test suite on Travis CI
Song Meta Data Mapping

Fixed the mapping of title to track field.  Added additional year and extension information from subsonic
Design is CRAP: Design principles for programmers
Add tests for numerically-based property types

There are some DataMapper property types that fail to get saved and reconstituted consistently (DateTime, symbolic and textual Enums). So, I added a few tests to show this.

I feel like there are perhaps some properties whose primitive types are not strings already that just need to get cast or something.
Most of the primitives for the failing types are numeric in nature.
Fixed bugs with wrapping yourself in a closet, new CentCom uniforms, reactive teleport armor now works and has a sprite, fixed implanter issues.
fix shadowing of outer for loop var i
Dev

Agregando una nueva imagen en el home y arreglando un bug en comparar
Better menu and ClearCache

I've added a better and cleaner way of getting the menu across the different plugins. Before I had to extend the controller, but now there isn't a need for it.

I've also implemented a ClearCache api from within the plugin, since there was a call to it, but it needed to be implemented for each controller.

The ClearCache lib is based on Ceeram's plugin and can be customized afterwards to delete only certain files.

Let me know if you run into any issues
Added docs to documentation repository (this should help with version control of these documents).

Adds all existing design, requirements and process control
documents to the repository, along with PDF copies of these
where possible.

Includes the following along with original author(s):

* Work Breakdown Structure (Dan Sweet)
* Project Plan (Dan Sweet)
* Requirements summary  (Minh Pham, Dan Sweet)
* Design Summary (Joseph Woodruff)
* Risk Management Plan (Joseph Woodruff)

Signed-off-by: Joseph Woodruff <jawoodru@cs.pdx.edu>
Updated to warn about turntable.fm's new rules.
Silent printf warning

Macro ```ThrowStringException``` is used by other macros that build printf-like functions with varied number of arguments. The attribute is meant to check the format, but since there are more than one, the check only pass in one of them. The zero tells GCC to only check the format string, not the arguments themselves, so it's not completely removing all checks.

Another way to fix this warning would be to implement ```ThrowStringExceptionN``` for N between nil and 4, but that throws away the benefit of using variadic arguments in the first place. I don't feel particularly strong about this patch, either way is fine.

The warnings were:

    /home/rengolin/workspace/hpcc/src/dali/ft/fttransform.cpp: In member function ‘void TransferServer::deserializeAction(MemoryBuffer&, unsigned int)’:
    /home/rengolin/workspace/hpcc/src/dali/ft/fttransform.cpp:583:1: warning: format ‘%lld’ expects type ‘long long int’, but argument 3 has type ‘const char*’
    /home/rengolin/workspace/hpcc/src/dali/ft/fttransform.cpp:583:1: warning: format ‘%lld’ expects type ‘long long int’, but argument 4 has type ‘const char*’
    /home/rengolin/workspace/hpcc/src/dali/ft/fttransform.cpp:583:1: warning: too few arguments for format
Rails 3.1 and Rails 3.2 support

The second part of https://github.com/kovyrin/db-charmer/pull/36
Added support for Dust (#8)

Added support for Dust rendering: https://github.com/visionmedia/consolidate.js/issues/8

I have not done extensive testing, but works with the examples from their homepage:

**index.dust**
```Hello {name}! You have {count} new messages.```

**Code:**

```javascript
var c = require('consolidate');
c.dust('path/to/index.dust', {"name": "foobar", "count": 5}, function(err, html) {
	console.log(html);
});
```

**Output:**
```Hello foobar! You have 5 new messages.```


On a side note, the latest version of Dust does not seem to work with newer Node as it uses require.paths, there's a fork that I used instead: https://github.com/semanticprogrammer/dustjs.git
Changed display of wind data

It doesn't make much sense displaying wind speed units (see [this screenshot](http://db.tt/Z2o0zy51)) if no wind data is available.
(I hope I haven't made any stupid syntax mistake!)
clean up and minor issues
Added Capes

I added a basic cape system to the viewer.  It's also setup so that it can be enabled to pull from the 3 most popular 3rd party cape providers too, but are disabled by default in this branch.

Thought you might want it.
possiblity to change default query
Russian translation for Visual Git Guide

Hello Mark. My name is Alex. I like your Visual Git Guide very much. And I've translated it into Russian. I hope you'll pull my translation for your guide. Thank you :)
Conversão da página "Mitos/FUD" close #4
Fix overriden partial path

Hey Brian,

The extension is not working anymore for me now, the overriden partial path must be updated.

Here it is.

Bests,

Richard
Remove dependency reader

Since DependencyReader doesn't do much work anymore I removed it completely.
updated diazo theme recipe, added missing image
Correctly parse nested hashes

I've put in a spec and fix for hashes that are 1 level deep (recursively call from_hstore once). 

This works well, but it goes a bit crazy if you start trying to go deeper. I'm happy to look into that some more in future, but for now I'd love it if this could be included in the gem.
Added 2 more FireGento modules to package list
Positioning imageView properly

Hi again, 

I've added imageView positioning example into the demo project. 
You can see it.

Regards
SungDong Kim
Use entities from the filelib project
[TIMOB-7133] Removed Ti.Media.Item and Ti.Media.AudioRecorder.
LPS-25021 - Web Content Display name overlaps configurations when scoped

Hey Nate,

Attached is the update to fix the web content display name when it is scoped. This issue can be found at http://issues.liferay.com/browse/LPS-25021. If this passes, can you please have Brian backport this change?

Please let me know if you have any questions. Thanks!

- Jon Mak
Fix `update()` method to merge `$data` & `$updates` before triggering error

`update()` method does not accept update data array (only $options) and so does not merge it with `$updates` class variable. This cause the error *"Nothing to update in Mongo collection or update is not an array"* whenever we use the method no matter what. This should address the issue.

P.S. I didn't check the `update_all()` method yet :p
transactionStart and transactionEnd have been replaced

transactionStart and transactionEnd have been replaced with startTransaction and endTransaction
Update init.rc to fix audio and telephony

Also switch to busybox's httpd
[TECHNICAL SUPPORT] LPS-25797

Liferay.Service object calls a method which might not have been loaded

Hi Sergio,

Small JS fix, tricky one. Please review it.

Thanks,

Máté
move project_status div over slightly so it doesn't overlap with change build info

The changeset build info will overlap with build status on occasion with the right settings.  I'll attach a screenshot to issue.
Hide shifts as soon as they begin

Hi Eric,

If we could get this out before the start of next term that would be great.

Thanks,

-Justin V
Smart store explorer

This is Smart Store Explorer, brought to Android.  One thing to note is that I manually added links to icudt46l.zip (to this and the SFDCAccounts project), because that file was not being copied...or at least it wasn't able to be read at runtime, which was causing SmartStore functionality not to work.  If there's another way to include these assets with the new library project, let me know.
Deshabilitar el IVA y valores actualizados

Con algunos clientes extranjeros no aplica el IVA, y establecerlo a 0
en las preferencias hacía salta el valor por defecto (18), por lo que
se ha añadido la posibilidad de deshabilitarlo al estilo del IRPF.

Además se han corregido los valores del IVA e IRPF por defecto para adaptarlos
a la reforma fiscal de 2012, quedando ambos valores en el 21%
Updates templates to more SS3-ready syntax

- Use $Name in if/loop
- Replace instances of control with loop
Testcase and fix for #JENKINS-12494

See https://issues.jenkins-ci.org/browse/JENKINS-12494
for unique Gorm constraints, created index should be created with unique: true
Handle initial textarea values

 * Make sure textareas expand to fit their original value once initialized
rm more non translated strings
Ovirt upstream rebased

Includes Replace Brick Start, Abort, Status, Pause and Commit.
Update models/ion_auth_model.php

I added in a reference to the identity key, which must be sent by the library.  I guess you could also load the ci class and re-fetch it, but this seemed cleaner.
SOS-823
First pass at typography

First thoughts about typography. Need to see more about Assets and Sass/SCSS to see what should be where.
正确的ubuntu安装方式

正确的ubuntu安装方式，见ubuntu官方https://wiki.ubuntu.com/Go
第十三章最末尾的章节名出错

改了几处错误,包括章节名和链接
Rewrited the last segment in 2.3; modified some sentence.
deleted supertest.

Supertest is gone.
quick restore
Next fixbug packager

- Issue - blackberry/BB10-Webworks-Packager#51
- Modified code in lib/bbwp.js, lib/ file-manager.js, lib/config-parser, so it's now works for option -s. It leaves src folder when there isn't any error in packager, and doesn't leave src folder when there is an error in packager.
Modified MGSplitViewController.m

Hi, XCode 4.2 gave me a warning whenever we compiled due to the datatype being NSObject. I changed it to UIViewController to satisfy the compiler.
DRY vim command; use inspect() to escape quotes

here you go
Update common-vendor.mk
Added a new error constant, SESSION_ALREADY_EXISTS to the blackberry.push.PushService class.

Added a new error constant and a unit test.
shadow.png fixes #3
Japanese translation of episode 322
Update zh-Hant localization strings.

As titled. I just translated all strings into zh-Hant.
Let me know if you have any concern. :)
add homepage so rubygems.org generates a link on the gem page making it ...

...easy to get here
Add link to Jetty HTTP Transport Plugin
OpenID and handlebars templates

So this is still pretty rough, I need to take the time to go back through and clean up the approach, but It seems to be mostly working, so I wanted to get it out there.

OpenID is implemented using passport, http://passportjs.org.

I had to abandon my static html approach for the moment, it was simpler to render the footer stuff server side.  I'm going to consider pushing that client side again in the future, but for now this seems to be a good solution.  I'm using handlebars templates (not using any features that couldn't be used with plain old mustache yet), for a pretty straight forward solution.
Clarify that __ctfe never causes code generation

Many people have been confused about this. The existing docs say
it evaluates to 'true' but don't spell out what that means.
Make ScreenGrab responsive to changes in window dimensions

Remove hard coded window size values and instead use
captureWidth = ofGetWidth() and captureHeight = ofGetHeight()
which might not be the way to do things in C++/oF but seemed
right at the time.
FixTime month bug
Play 2

I didn’t had much time this week, I only rebased on branch dev and changed the build so you don’t need to build play from sources anymore. Just hit: `sbt start` and you’re gone (you need [sbt](https://github.com/harrah/xsbt/wiki), obviously)
chromosomy!!
On 09/01/2012 21:45, Alejandro Revilla wrote:
> Well, it's actually not a big deal, just a new thread would be created to service the request (which is good), still under the limits of the ThreadPool max size, so I guess we are good to go.
> 

Ok, so nothing fatal, great.

-- 
Mark
Hi,

The code is fine, even if I'm the big fan of putting commas before the parameters, but It's just me.
Ok, great. I will merge this as is, but we should definitely address the first comment.
Není třeba opravit ještě testy, např: https://github.com/nette/nette/blob/master/tests/Nette/Latte/UIMacros.renderSnippets.phpt?
s/<=/</
see my comments in general thread?
BTW. Do you mind commenting on the diff page rather then per-commit page. This will make it easier to answer as they will show up on the discussion page.
Chyba je u Nette\Utils\LimitedScope::load(NETTE_DIR . substr($type, 5) . '.php', TRUE); - zpětná lomítka linuxový/macový filesystém nebere, mělo by se to nahrazovat za /, případně DIRECTORY_SEPARATOR. (Zpětná lomítka tam zůstávají z namespace.)
At least it's still better than before when it was advised to have different indentation for different methods :ok_hand:

Considering it a small victory :smirk:
Thanks for fixing this. I wasted a few hours trying to deploy a simple Sinatra app to Heroku that refused to connect to the shared database.

I kept getting this error but couldn't figure out why:
ActiveRecord::StatementInvalid: PGError: ERROR:  invalid value for parameter "client_min_messages": ""

Since I wasn't setting a value for :min_messages, it was being set to "", and hence the error.
Actually, the core.php got _copied from the repo, deleted from the clone and symlink-ed back into the **shared_path/config/core.php**_ which allowed to have a separate core config file on your local repo and your production box. Same as for the database. I find this very convenient so that I can have debug level 2 while coding, and don't have to think about switching back to 0 before deploying. Maybe you have another trick?
Very nice! 

Can you add an example to examples/example.rb and update the README.md#examples to explain how to use boolean values?
It makes a great final integration test for us as library developers, and ensures that folks have a clear example of how to use it.
The license should then be changed to MIT?

Another reason to keep all symfony-derived code under MIT is so that we can upstream our changes should any be useful.
I kinda played around with it and then left it out as not-really-that-useful in the spirit of YAGNI. I'll merge with HEAD and sent you a pull request if you want
Voisi laittaa logout paketin, jos ollaan pelissä.
test merge hub
Not to say this won't work at all...it just needs the version in the path. I think the `MACRUBY_VERSION.to_f` would be safer to use than than `RUBY_VERSION`. At least at this point with how MacRuby is changing still.
md5 seems to be wrong.
Should be 6ce61f71f2ec199f1a98012eb0f960bf.
You have to be careful doing that - that means that the variables have internal linkage and thus means it may or may not take up more memory.  If the compiler is smart, it will use it to inline values and evaluate constant expressions, then the linker will take out the duplicate variables.  However, if it's stupid, it will keep all the duplicates.  Only with ints does the standard mandate that the compiler has to inline them.  And even then, you need to have a declaration in one translation unit or you will get undefined external symbol errors.  Just saying, tread carefully.
It's in UpDown.hs or some such file.
Ah, reading the ruby-mysql documentation, it appears that it defaults to true there.  You should probably chance the documentation to remove the Default is true statement, since Sequel has no default (and ruby-mysql could theoretically change it's default).  Also, use opts.has_key?(:query_with_result) instead of checking if the value is nil.

Most of my other questions still stand.  Is the adapter usable after this is disabled?  What are the positive/negative effects of disabling it?  Why does disabling this setting help your use case of streaming large datasets?
@haugstrup I think we should change this to take just options = {} as an argument - that's more in line with what we do elsewhere, and also prettier when there is more than one optional parameter.
Thanks for pointing me to there.  Still learning my way around the source tree. I'll add some tests for the feature there.
This fix is great!

One minor piece of feedback.  You can use SnapshotLineUtil.IsBlankLine to determine if a line is blank vs. using Trim().
Sorry about that. That bit of code is a remnant of how I initially thought I was going to implement this solution. I've removed it with my latest commit.
@Reamer 
Show me you original issuse...
Ah, yes! That explains the spread through the all the UNIXy dynamic languages...
I think the preConfiguration url is built from IsoSpec, so it shouldn't be here, relationship wise
I enabled it, since i thought that we are always low on RAM. We are on Linux, and its just caching ram. For linux newbs like me, it looked like we are always out of it :p
Okay, I've reverted this change, so readS32() now just calls readU32() again.
Does this mean, perhaps, that readU32() is not actually correct?  Or is it actually only supposed to be capable of unsigned 31 bit ints?
oh yeah, huh, I wonder why that's not a syntax error
@twobraids Adding this. Wait for the next commit. 

@AdrianGaudebert Strange as it may seem, `datetime.datetime.utcnow()` is actually very different from `datetime.datetime.now(datetimeutil.UTC)`. You can't even compare them. The HOUR part on them will always be identical but they're still very different things.
really cool and powerful idea.
不是，是我一同学，他设计了不少女性武将。
it would be better to change CPortAudioRecord in line 243:

public SRecordDevice[] RecordDevices()
{
   if (!_initialized)
      return null;

   if (_Devices.Count == 0)
      return null;

   return _Devices.ToArray();
}


Then _DeviceNr would be always -1 and all of these changes in this file will not be necessary
Node attributes in chef _are_ global.

I am not sure this sort of detection will work as expected. We may use one
of two different keys for each case, but do not allow to mix them.

--
Nikolay Sokolov
Software Engineer
Grid Dynamics
24.02.2012 13:47 пользователь "Oleg Aravin" <
reply@reply.github.com>
написал:

> Do you mean to add some optional specific parameters for recipes and use
> parameters as global for all recipes?
>    runlist: 'recipe[mysql, yoursql]'
>    parameters:
>      tunable:
>        max_allowed_packet: 1
>    recipeParameters:
>      mysql:
>        value: 42
>      yoursql:
>        value: 43
>
> Maybe it will be good to use just  "parameters: <params>" if user
> specified only one recipe and use "parameters: <recipe> : <params>" if user
> specified more than one recipe
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/erkfel/manifests/commit/18d4b3681b061bbbb819fabcd53af4e1e93905b9#commitcomment-1006691
>
thanks for working on this, Erik! 

tests would be great. note that there's an existing test that simulates user key responses in order to test a small version of the Stroop demo. its testRun_FastStroopPsyExp() in testTheApp/testbuilder/testExperiment.py
Thanks for your answer, I'm going to test it.
Uwe, what sort of things could I do to check the stlinkv1 here?  try and write a program to SRAM that's greater than 0x1800 or so, see if it still works?
This script hasn't been converted yet, has it?
and this change would be unnecessary
Using my great Italian skills (read: Google Translate) I would agree with you here. Would you mind submitting a patch to fix this issue?
also where do you use this?
Whoops. Sorry. I suppose that's the cost of only testing it on my profile.

On Tue, Feb 21, 2012 at 1:17 PM, Rob Renaud <
reply@reply.github.com
> wrote:

> this has a bug, you need to get the deck with the unnormalized name,
> target_player_cur_name
>
> fix is here:
>
>
> https://github.com/rrenaud/dominionstats/commit/70d940cdadd4fa6a45bb73587e09bb8842ed26e6
>
> and here (cause i was too brazen)
>
>
> https://github.com/rrenaud/dominionstats/commit/d482bd6fabf05eada263b8f2bbe30d51259f53d5
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/DLu/dominionstats/commit/6374e9558463c658c161967d6322930d81e7d372#commitcomment-992738
>
Nice, this is handy.
r+ code looks good, with chat in irc
you could actually use getMapper("dozerBeanMapping.xml") instead
Good。This is my needed。
You should remove this file as it was used to test the merge options
:( I'd rather like to see it replaced, not removed. Now it's a clear regression.
Thanks for the catch. Fixed in 8129412155caa9abe747518c7dfe278331e91695
This would be better as:

"expected #{selector_name} to be returned #{@options[:count]} times; it was returned [count] times"
mmakc a écrit :
> I was getting mount failures on NFSv3 - mount would return a filehandle, FSINFO with the same filehandle will fail
>
>   
What kind of options ? I have use FSAL_XFS hundreds of times (it was my 
default FSAL to compile with at cthon because it runs easily on my 
laptop) and met no issue of that kind.
Did you configure the xfs file system in a specific way ?
These fixtures are missing the required headers.
CSS and JS files are aggregated so while loading different files on different pages would resume bandwidth required it might render server and local caching less useful.
I am following a convention to call `test :expand_path` with an atom with are unit testing methods. :)
@md-5 well it's because every admincmd command are done in separated thread. Meaning accessing a resource that is not thread safe, like a Minecraft world, can pose some problems. Example, another plugin is removing items dropped by checking all entities, if we add an entity in the list while the other plugin is iterating on it, it will throw a concurrency exception and make the server crash.
Yes, this is correct (see "A Note on the 'Game' point" in the CinchRules document).  While we're on the subject, it may be useful to create a Wiki page for the rules in addition to the .md file- maybe I'll get around to it sometime.
Understandable. :-)
This creates a bug. 

To reproduce, I have the following code
```
class Report {
/**
	 * The owner of this report
	 * @var entities\User | null
	 * @ManyToOne(targetEntity="User")
	 * @JoinColumn(name="owner", referencedColumnName="uid")
	 * @Id
	 */
	private $owner;

```
If Report is fetched, and owner is null, then we get an undefined index warning on Line 2288 in UnitOfWork.php
```
foreach ($class->identifier as $fieldName) {
                $id[$fieldName] = isset($class->associationMappings[$fieldName])
                    ? $data[$class->associationMappings[$fieldName]['joinColumns'][0]['name']]  // <---- HERE 
                    : $data[$fieldName];
            }
```
The $rowData variable in gatherRowData function that this commit modifies will be returned without mapped fields which are null. Hence, the above exception. 
The bug is not critical, however, it will generate warnings.
sasaSSAssadsad
Should work. Syntax highlight, however, is a github feature.
man i love bonfire but i cant say what is going in my heart for bonfire because i cant speack english whell :(
yes, in mangosR2 and mangos the range is a little bad :S
что за метод allCollectors?
сколько рза напоминать про названия методов? у меня скоро кончится терпение
Fair enough. This is far from done. I wanted to push to let you fix everything for me :)
Ha, that's hilarious. Good find!
Hi Dan,

Here's the 2 stack traces of the failing tests.

TestCase 'XeroApi.Tests.SignatureTests.TestSignAccessToken'
failed: System.Security.Cryptography.CryptographicException : The specified
network password is not correct.
at
System.Security.Cryptography.CryptographicException.ThrowCryptographicException(Int32
hr)
at
System.Security.Cryptography.X509Certificates.X509Utils._LoadCertFromFile(String
fileName, IntPtr password, UInt32 dwFlags, Boolean persistKeySet,
SafeCertContextHandle& pCertCtx)
at
System.Security.Cryptography.X509Certificates.X509Certificate.LoadCertificateFromFile(String
fileName, Object password, X509KeyStorageFlags keyStorageFlags)
at
System.Security.Cryptography.X509Certificates.X509Certificate2..ctor(String
fileName)
SignatureTests.cs(65,0): at
XeroApi.Tests.SignatureTests.TestSignAccessToken()

TestCase 'XeroApi.Tests.SignatureTests.TestSignRequestToken'
failed: System.Security.Cryptography.CryptographicException : The specified
network password is not correct.
at
System.Security.Cryptography.CryptographicException.ThrowCryptographicException(Int32
hr)
at
System.Security.Cryptography.X509Certificates.X509Utils._LoadCertFromFile(String
fileName, IntPtr password, UInt32 dwFlags, Boolean persistKeySet,
SafeCertContextHandle& pCertCtx)
at
System.Security.Cryptography.X509Certificates.X509Certificate.LoadCertificateFromFile(String
fileName, Object password, X509KeyStorageFlags keyStorageFlags)
at
System.Security.Cryptography.X509Certificates.X509Certificate2..ctor(String
fileName)
SignatureTests.cs(20,0): at
XeroApi.Tests.SignatureTests.TestSignRequestToken()

On Wed, Jan 18, 2012 at 10:48 AM, Dan Barratt <
reply@reply.github.com
> wrote:

> Hi @meboz,
>
> The build servers that I'm using aren't reporting and build failures. Can
> you tell me which unit tests are failing?
>
> Dan..
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/XeroAPI/XeroAPI.Net/commit/fe8d72dc2cdd252ebeff08520bf0d1dd7623993b#commitcomment-876384
>



-- 
Cheers,
Byron

======================
p: 02 4384 5414
The context isn't used anyway and no, the buffer is not limited in size. One solution, Rewrite sc_hex_dump:

from
void sc_hex_dump(struct sc_context *ctx, int level, const u8 * in, size_t count, char *buf, size_t len)
to
void _sc_hex_dump(const u8 * in, size_t count, char *buf, size_t len)

and for backward compatibility use
#define sc_hex_dump(ctx, level, in, count, buf, len) _sc_hex_dump(in, count, buf, len)

Now use _sc_hex_dump to format your buffer in pkcs11-display.c and print it by hand. While you're at it you can rewrite sc_dump_hex to use _sc_hex_dump instead.
```
undefined method `truncate_html' for #<#<Class:0x007fb86a45e438>:0x007fb86a462830>
```

I guess it's used in other plugins too ;-) that's okay, they can just require it themselves
Qual o objetivo desse diretório assets? Se não houver um bom motivo, manteremos o css, js e img em /public mesmo.

Além disso, esse main.js não tem nada, então nem deveria estar nesse pull request.
Of course, the `Unstage()` method would be updated accordingly...
... and the search/query part of the url changes in the browser, then the `$routeUpdate` event....
I can't believe I'm getting pull requests for this. :)

That said, this will get spaceShip.points and the contents of the "score" div out of sync.
Could we call it something other than slow: bulk, request_finished or bulk or something even though the packets get sent later, it might not be slower.
What is the query that you are trying to parse?

On Thu, Jan 5, 2012 at 5:31 PM, recalcitrant <
reply@reply.github.com
> wrote:

> The * function returns a ResultSetParser but the ~ function expects a
> RowParser.
>
> So how can one combine an object with a Sequence of other objects as
> before the refactoring?
>
> This worked before : User.single ~ ( Permission.single * )
> This does not anymore : User.single ~ ( Permission.single * )
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/playframework/Play20/commit/f35fd305d98038ab0fe3f8076368ee7f8d90015b#commitcomment-840836
>



-- 
www.sadekdrobi.com
ʎdoɹʇuǝ
What are your reasons for this hack? I am wondering if we need to provide some facilities but I am not sure of your intent.

Ideally isolate this type of code change to a separate commit in Git so that we can easily surgically remove it.
Correction, *two* typos :)

sha: f86ffb5fe4bd96c7c4a8867bfe8aa5836dcb0d93
@jprichard Our esp has a limit for an HTTP request: 8M bytes.
I must admit that I did not a full analysis/refactoring of this. This activity can be called with id and gecode as input in onCreate but in this context this is obviously not the case in this context.
I promise I will do this.
Это совсем правильно, если не хочешь мерцания для английского. Вполне разумное поведению по умолчанию. Зачем добавлять ноды скриптом если они уже есть? :)
Already done in another commit, see § https://github.com/framiere/rt-analytics/commit/613e3e5cbf19b4d69833aeba730f7940330a70bd#L3R78
exactly, I have no clue. I'm pretty sure tests where not failing on 3.4 when doing the releases, still even if I rollback all changes back to 3.4.0.Final one test fails for me now.
Are you sure you will use this patch on other kernel trees?

So it is better remove the ifdef for kernelversions.
No, that seemed like a bug, unlikley anything relying on it.
Could you please explain what this change does?
A popular subtitle editor in China doesn't read UTF-8, only UTF-16 and system code page. Additionally, several versions of VSFilter can't correctly handle UTF-8 subtitles (Chinese characters in wrong fonts etc.). So I always save subtitles in UTF-16.
Name this `-ks_normalizedURL` I'd say
I've commented on your comments! It's comments all the way down!
Vale, por un momento me asustaron tus comentarios, hare los cambios indicados. Se me han escapado unos cuantos `console.log` innecesarios. Y una cosa más estas lineas [315-318](https://github.com/nodejs-mexico/nomi-nation/blob/master/routes/dashboard.js#L315-318) y la [337-340](https://github.com/nodejs-mexico/nomi-nation/blob/master/routes/dashboard.js#L337-340) es lo mismo, tienen alguna función en especifico? ¿Leiste el nuevo wiki que te parece?
Bien los módulos, con RequireJS.
Faltaría agregar Director como dependencia en Movie, como se enuncia en Day7/Practice 3.
The pylibmc package is faster in most benchmarks, but it does depend on an external C lib so it's harder to install.
Let's match the pre-existing behavior for now and clean it up once we fully replace the Flex graph widget.
By "happen" I mean "completely finish".
It won't fix the problem where "space" and "shift" are completely missing, though
Does commented out rule solve any problems?
You should open the file and put all the IDs in a set outside the while loop. Unless you want to add to it in case the file changes, but there are ways to tell when a file changes without reading it in.
Thanks to Tim Ryan for teaching me about lexical scope!
https://github.com/agoddard/rambling-hipster/commit/94f80f027d20aac3c56a00f7c81457159425e226
Sorry, it was added with the commit from my fork
I think this would be a forward slash / rather than backslash \
Great! We can add the issue status here.
Do we need to include other information?
Arghhh I've done it in similar way:

```csharp

			    var exception = Assert.Throws<GenericADOException>(() =>
			    {
                    session.Save(e1);
                    session.Save(e2);
                    session.Flush();
                });

			    Assert.That(exception.InnerException, Is.AssignableTo<DbException>()
			                                              .With.Message.StringContaining("unique").IgnoreCase
			                                              .And.Message.StringContaining("constraint").IgnoreCase);
```

And anable it for other dialects (I've checked on Oracle and SQLite)
Why not raising an exception when `require "json"` fails on 1.8? Why force everyone using the stable version of Ruby (1.9) to install a gem?
Well, my college newspaper, http://badgerherald.com/, is in the process of a redesign and are interested in using Field Day on their current MT5 install. For photos at first, but who knows what later. So that can be the first major test. The installation has 41,000 entries and the site gets about 600-700K views monthly.
hm why you didnt add your name for copyright?
Cool!  :D
for tethering
nothing.
anyway,I saw that cp.sh in fserve sources
but it's really useful it saves lot of time.I didn't sign my kernel.
but when u r going to publish it.
Hm.. Here is my real working code of both client and server handlers and it does not use the io_service<pre><code>boost::shared_ptr<boost::asio::ssl::context> CWebSocketServerHandler::on_tls_init(void) {
	// create a tls context, init, and return.
	boost::shared_ptr<boost::asio::ssl::context> context(new boost::asio::ssl::context(boost::asio::ssl::context::tlsv1));
	try {
		context->set_options(boost::asio::ssl::context::default_workarounds |
			boost::asio::ssl::context::no_sslv2 |
			boost::asio::ssl::context::single_dh_use);
		context->set_password_callback(boost::bind(&type::get_password, this));
		context->use_certificate_chain_file(m_strCAFilePath);
		context->use_private_key_file(m_strKeyFilePath, boost::asio::ssl::context::pem);
	} catch (std::exception& e) {
		std::cout << e.what() << std::endl;
	}
	return context;
}

boost::shared_ptr<boost::asio::ssl::context> CWebSocketClientHandler::on_tls_init(void) {
	// create a tls context, init, and return.
	boost::shared_ptr<boost::asio::ssl::context> context(new boost::asio::ssl::context(boost::asio::ssl::context::tlsv1));
	try {
		context->set_verify_mode(boost::asio::ssl::context::verify_peer);
		context->load_verify_file(m_strCAFilePath);
	} catch (std::exception& e) {
		std::cout << e.what() << std::endl;
	}
	return context;
}</code></pre>
Please don't commit this kind of whitespace change. It's impossible to see what you've actually changed without diving into tools that are whitespace-aware.
Just delete the code instead of commenting it out. Git has history :)
Seems pretty straightforward.  Looks good to me.
On Feb 4, 2012, at 7:30 PM, Peter Kleiweg wrote:
> Nee, echt niet. De Accept-header is bedoeld om de beperkingen en voorkeuren van een client aan te geven, onafhankelijk van individuele requests. 

Dat spreekt de standaard je gewoon tegen, die expliciet zegt dat een 'Accept' header gebruikt moet worden om het 'formaat' van de daarop volgende request aan te geven. Bovendien is dit hoe vrijwel elke web framework het doet. Bijvoorbeeld:

http://apidock.com/rails/ActionController/MimeResponds/respond_to
http://www.yesodweb.com/book/rest

> Een gemiddelde webbrowser, bijvoorbeeld. 

alpinocorpus-server is een server voor applicaties, niet voor mensen om rechtstreeks in hun browser te gebruiken.

Anyway, na de argumenten gehoord te hebben, besluit ik me toch netjes aan de standaard te houden. Dus in de toekomst zal ik de server aanpassen zodat de 'accepts' header gehonoreerd wordt.

Groeten,
Daniël

Ps. Graag alpinocorpus/dact discussies om aangegeven redenen in het Engels voeren. Ik hoop Dact in de toekomst ook voor Engelstalige treebanks geschikt te maken, en dan kunnen er meer internationale mensen bij het project betrokken zijn. Het is dan praktisch als de geschiedenis van het project voor hun leesbaar is.
No próximo commit vou tirar esses "page.should_not have_content 'Disciplina was successfully created.'" Esse teste nunca vai quebrar, essa mensagem só aparece depois de você salvar o objeto não quando você visita a pagina de edição/criação. poderia testar "page.should have_content algum_campo_do_form" mas acho desnecessário já que o teste já quebraria por não existir a mensagem pedindo para logar.
Now this will only be sent once?
icons looks nicer, and smaller is better for the X8, but the select button, should maybe looks like something more clear like "OK"
Yes, I can confirm that the repo seems to contain the caliper jar now. The weird thing is that that same directory was empty when I opened the ticket. I made some noise in a few places, maybe someone updated it? Or maybe it was some kind of glitch?

I don't know why I thought sbt11 wasn't working.

Anyway, thanks for responding. I can confirm that sbt11 works fine and the repo does exist now. Sorry for the noise... I am still not sure why it was missing earlier.
@neoascetic у тебя в этом посте работают комментарии: http://vladislavbyk.github.com/install-cdrtools-in-ubuntu.html? Вроде поправил, а все равно не работают. Хотя в первом посте все ок. Кэш тоже почистил.
Break  ,      .
          - 
.

     -    .
07.03.2012 12:08  "ugnich" <
reply@reply.github.com>
:

>  mediaOut  return ret.
>   getUserOptionInt.
>   ,    i  added   ,
>    C. , -, break      for.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/alexandrdmitrievich/com.juick.http.www/commit/6a56ddd955724555cc12d2c422b87abe7bdff01a#commitcomment-1052238
>
adridu59 - i tried for me same problem
please use A=( "${A[@]}" "$2" )
I'm pretty sure you cannot have two `wait()` calls like this and they need to be nested. Maybe that's the issue?
Was removing the comments here intentional?
Again, here make it clear that the tuple is expected to have 2 entries in it?
Kind of confused here. We are trying to heartbeat the unreachable members on the lines below. What is the FIXME referring to?

I think we need to try to talk to unreachable nodes until they are downed and removed.
If it's a throw-away instance we don't care, and if it isn't, we don't want to race with others doing the same thing.
You are closing over sender here, in a future callback
Yeah, a man can change, right? :-)
Problem solved:
    protected val counter = MTF.tfCounter
Don't use default params AND aux ctor
damn java compatibility, I have to move the up one level, and remove the nice `Fields` grouping (the idea was to support `import Cpu.Fields._`)
Hmm, let’s see whether we need that; I’ll keep the TODO
I'm working on it.  I suspect we may need to keep it around for a while so that migrations will run.  I've had similar with the Cardtype model.
Arf, i didn't see that github interpreted the underscore. Yes, i was referring about `__all__`.
Why are we supporting this custom masked view instead of the built-in NumPy version?
`self` is not aligned with `gtab` and the other parameters.
I'm big -1 on IE 7 support on CKAN core, the user share simply does not justify the pain.
That doesn't mean that if support for IE 7 for a particular project is needed, we may offer it for an extra cost.
This is excellent. I was hoping we'd be able to do it in a single database call and not destroy the DB in the process of looking these up.
Same for "fully intend to". I agree with mentioning it'll be improved in the future, but the tone of this and the above comment sounds like the writer has a grudge against Pulp.
I'm not quite sure how well it will work out.  Some runs indicated it could be really fast, but something happened recently to slow it down.

I now write both keys to local cache when I can, so many of the id fetches should hit there.  If they do fail, they fail once and do a key lookup (cache and DB as needed) and store as id (if it has one)
I think it returns the @account object, a User, but I'll double check that.

@account is almost always the associated object, not the card.
DOPE.
Fixed with new patch.
I'd really like inOrigin moved out of the getXMLHttpRequest call, since as it is, the function will be declared again for each call to getXMLHttpRequest.  I think it could just be another protected method without a problem.
The `int` cast here should be removed.
`ifndef` win32, maybe we should just set all of these to `false`, rather than leaving them undefined?
Ha!  Agree with you there but it is only a line.  If the relative path for this subproject ever changes, having an explicit path helps.
Take it this is putting back in the `cylc` args fix - somehow been lost in previous merges
is "print scheme" needed when there's an Exception raised?
лишнюю строчку нужно убрать
QtCore/QDir
Why not use `to_sentence`? I believe some languages the two_words_connector cannot be used to "connect" different words.
Right. Too much Python lately.
Don't hand roll FatalError; call abort.
indentation is off here, too. (tabs vs spaces, perhaps?)
More decimals could be useful to have here.
Just do `linear_momentum_sys += e.linear_momentum(frame)`.  There's no need to create the intermediate variable that is not used again.
`yield` is fancy enough for me at the moment. I'm at my limit of complexity with this branch.
Try doing the following

```
import random
random.shuffle(terms)
```

and run the tests a few times.  This will hopefully tell you a little more about what kinds of errors can happen, based on functions that expect certain arg ordering.
Hm, that's right.  However, in the majority of textbooks I've seen (including an edition of McLane's "Categories for the Working Mathematician"), hom-classes are _defined_ to be sets.  I will indeed stick with what I have now, but I'll keep the term "locally-small" in mind.  Thanks for pointing out!
this docstring example should now be deleted, I believe.
This looked better before.
Also the `:` is not usually part of the section title, i.e.
```
Examples
========
```
There are many instances where the above two comments apply, and I can't tell quickly what's new from what's moved, so I though I'll just mention them anyway.
Avoid using select if you don't have to.  It's better to do:

    local _, _, _, _, class = GetRaidRosterInfo(unit)
I changed it so it's different to info. I don't think two logs should be the same color. White may not be the best though
Nice one! :P

Btw, man this is a **huge** pull-request! :D
Because we're no-longer using the (now removed) akka registry for naming actors, instead relying on our own implementation to map names to actors. 

Still- it's useful to have it in the logs, so I'll add it back in to the createPairActor methods.
Do queries "use" indexes? Or do they use keys which may or may not be indexed?
Do we need this `<dl>` here, if its contents are just `&nbsp;`'s? Is it there to keep the layout flowing right?
This is weirder than I thought; the code here is actually straight copy/paste out of what used to be in save(), and presumably worked there -- or did it?
Should perhaps only initialize it if plupload is being used.
Needs a space between `(int)` and `$row`
Unless it's copy-pasted watch code in which case I'm assuming it works because it is not in a transaction.
bitfield?
please use double quotes in the outer part and normal quotes on the inner part to avoid \'
No, root directories are always style directories. Templates directory is a subdirectory, stored in $template_path. Should I add variable $template_path to interface?
Keep it as is for now.
In this case, "dbal class object" is fine.
I am not convinced that this is a good thing. What class names are going to come out of extensions? If you cannot have dots in config names, only replace dots with a legal character.
`$null` is a useless variable.