Hi, as I remember we said we will try if redirecting is better than staying
in the same editor window, so the user can get an idea about the different
opportunities.

2015-06-15 12:48 GMT+02:00 Laurent Pugin <notifications@github.com>:

> Didn't we agree we don't want to redirect to show after saving?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rism-ch/muscat/commit/ecbf52048990c319a21abc39957f017a7afc4908#commitcomment-11677647>
> .
>

--

Répertoire International des Sources Musicales
Stephan Hirsch
IT-Manager
Zentralredaktion
Sophienstraße 26
D-60487 Frankfurt am Main
Tel: +49 (0) 69 70 62 31
Fax: +49 (0) 69 70 60 26
http://www.rism.info


Hello,

this is a valuable contribution to handle edge cases / exceptions. I wouldn't set the value to "anonymous" but to null though. Then the DB call to list the charge points for outgoing requests will filter these out [1].

And I wonder how you can use this charge point afterwards, when you cannot send any requests to it and communication can only occur in one direction. For analysis maybe?

[1] https://github.com/RWTH-i5-IDSG/steve/blob/master/src/main/java/de/rwth/idsg/steve/repository/ChargePointRepositoryImpl.java#L61


Maybe, rename it to putStrLn?


Maybe, rename it to readLn?


Initial commit


I think I will add more box debris when i have time... :)


前一个才是啦……


Just curious, why was `IOW` chosen as a name?


This is still nonfunctional... or does this still needed ideas to be implemented? :)


yes, but it is not tested good enough.

On Mon, Jun 15, 2015 at 1:44 PM, Markus Staab <notifications@github.com>
wrote:

> will the CLI also benefit from this cache?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/php/php-src/commit/ea14c20ffb3f4a9e24f27ace81f70b2b0025ae15#commitcomment-11677572>
> .
>


There should not be unit directories in var_lib_t directories.

I would label systemd unit directories and symlinks always with a generic systemd unit type (for example systemd_unit_t) and never with a private systemd unit type.

This will simplify things , especially with regards to labeling.

That means that i would not use "systemdunitfile"  here


Thanks.  Updated.


Ok, let's try it like this then


What is the release schedule for this plugin? When is this change going to be released? We need this change for our project.


Thanks @mgruner,

I like this part: \$('ul#Activities li:contains($ActivityRandom)').length". I haven't known this before. Great!
I believe it could be used for improvement of some previous test and in further Selenium tests as well.

Regards
Zoran


@matDobek why would you want to do that? From what I understand, this could mean we can't access transactions happening between the last tx in current receipt.iap and the last tx of the last_receipt_info.


very nice


`try onReplayError(cause) finally onReplayComplete()` ?


Let me know if there is any industry accepted terminology I should use for the PES and PET behavior.


Might as well tidy this line

```diff
- @cached_body = File.open(self.filepath, "rb" ){ |file| file.read }
+ @cached_body = File.open(filepath, "rb") { |file| file.read }
```


SDSSS


- Ihnen fehlt die Berechtigung, um die diese Seite anzuzeigen.
+ Ihnen fehlt die Berechtigung, um diese Seite anzuzeigen.


So I believe, we just need to remove the "../"? I will make commit a temporary fix for this.


Please, generate a meaningful serialVersionUID (use your IDE to generate it), 1L does not make any sense.


Should this be `def body_as_text_scrubbed?`?


cuidado com o merge :warning:


NICE DURDENT


Thanks. Is it working on FreeBSD now?


> What's the reason for this change?

`ass_utils.h` is already included almost everywhere, so I probably thought it might possibly be a good place where to put things relevant to portability. Normally they’d go into config.h, but we can’t control that… wait.

We could put things relevant to portability into config.h.in, couldn’t we? We don’t have one at the moment, but we could add one if I understand correctly.


@rocapal I have added support for checking whether the qwt is of version 5 or not.


Referred Links:
https://www.twilio.com/blog/2014/02/twilio-on-rails-integrating-twilio-with-your-rails-4-app.html :+1:
https://www.twilio.com/docs/quickstart/ruby/client/incoming-calls


good


I hope this is the right version of nlopt - wasn't there a reason why it is included in treePL? Maybe patched?


Great!


{ in new line please


Shouldn't this be add_task_in_thread?


fdzhsfhsgfhgfh


dsgbsabvdf


It was removed some time ago. The light object uses only the color component information, so I save the components separately.


See ales-erjavec/orange3@f06fe08941aaeec787a57c8540c887eec3264547


Please do :-)


ok thik


Congrats on your persistence and for fixing it! We can dissect this in today's call.


- toolbar element removed as not needed since close button template and styling was updated for accessibility.
- hard coded colors removed and variables updated.
- font styles moved out into fonts.less (shares styling with buttons for consistency).


right


#13 , fine? :smile:


Check respawn delay mobs spawned on Hellbound (Naia mob 0 respawn delay)


s/Wraping/Wrapping


lol


Hm, not sure if it's a good idea to place such things into a template for an autogenerated file.

I don't know, using ass_utils.h feels somehow "wrong". But of course feelings don't really make a good argument. On the other hand, just including a portability header everywhere shouldn't be too much of an issue.


Nice :+1:


Yes, I noticed as well that this was for a reason; it was reverted in commit 7bd4f534b170755d6e9658b0ab9f2b382b9eaf9d. The same holds for `adolc`; installation of these two take quite a long time.


Ditch the comment - its turned stale!


learning design-pattern


might as well remove commented out reference to function that no longer exists


Good point, will do.


I read the message line of these specs as:

```diff
    describe :body do

        it 'returns a binary encoded string when newly created' do
-           foi_attachment = FactoryGirl.create(:body_text)
+           foi_attachment = FactoryGirl.build(:body_text)
            expect(foi_attachment.body.encoding.to_s).to eq('ASCII-8BIT')
        end

        it 'returns a binary encoded string when saved' do
            foi_attachment = FactoryGirl.create(:body_text)
-           foi_attachment = FoiAttachment.find(foi_attachment)
+           foi_attachment = FoiAttachment.find(foi_attachment.id)
            expect(foi_attachment.body.encoding.to_s).to eq('ASCII-8BIT')
        end

    end
```

If the current implementation is correct, I'm not sure I understand what we're testing in the second spec.


Here were volunteer_shift.start_time and volunteer_shift.end_time. As shown by the new unit test test_generate_report, it does not work, we need  volunteer_shift.shift.start_time and volunteer_shift.shift.end_time


I was thinking of this from the first time I saw it.


I think a lot of this code could be reduced as it's just a repeat of the components default styling. I would just keep in code which either overrides the default styling or editable by variables.

+1 once reduced


Bedankt man


Well, it's not possible in the True Great C, but it is in C99 :]


//这是POST请求
++(void)setaddPOSTLoadingUrl:(NSString *)Url Dictionary:(NSDictionary *)Dictionary success:(HttpSuccessOpen)Open;
//这是GET请求
++(void)setaddGETNetworkParametersUrl:(NSString *)Url  success:(HttpGETrequest)returnDic;


This commit does not work with Debian Wheezy.
err: Could not retrieve catalog from remote server: Error 400 on SERVER: right operand of + is not a number
at /etc/puppet/environments/bfproto/modules/sudo/manifests/params.pp:13 on node foo


TeamCity NEventStore :: MongoDB :: Build [Build 20](http://build.prxm.it:8811/viewLog.html?buildId=3019&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Runner failure (new) Build time: 00:01:36


I wonder if this should return the value object mentioned in https://github.com/mysociety/alaveteli/commit/d869d1173483d9423e6f61f5872ed17b87269940#commitcomment-11677735. Doesn't seem nice to store the state of this conversion in `body_as_text_scrubbed` in the `FoiAttachment` object.


//这是POST请求
++(void)setaddPOSTLoadingUrl:(NSString *)Url Dictionary:(NSDictionary *)Dictionary success:(HttpSuccessOpen)Open;
//这是GET请求
++(void)setaddGETNetworkParametersUrl:(NSString *)Url success:(HttpGETrequest)returnDic;


but cant we open ports manually in the security group settings?


Oh, right! Yeah, I got it right, I think, but I was just confused about the `into: :angular_validations` vs. the `into: :angular_directives` options for the output in ´translate_rails_validation`. I read that this was called with the `:angular_validations` option, which would've caused the behaviour I specified, I think, but it is instead being called with `:angular_directives`, picking the last, which in the case of _minimum length_ is the `ng-minlength` directive. Got it now. Score!

Great stuff, man! :+1:


**from datetime import datetime** outputs TypeError:
 descriptor 'time' of 'datetime.datetime' object needs an argument


expires_at, hidden and disabled columns were added by mistake. I reverted them in 6e068af7b so the overall diff is ok.


from datetime import datetime outputs TypeError:
descriptor 'time' of 'datetime.datetime' object needs an argument


showing system gpa


let us move this to a config file, instead of hard coding it inside the repo


Where's the problem?


que pontos são esses? Porto Alegre?


Pq não utiliza o jQuery dos bundles que vêm com o ASP.NET?


再帰を学びましょう！


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/13425/testReport/) for this commit**


It does. The '0 +' thing is supposed to take $::operatingsystemmajrelease and make it a number so it can be compared (>= 7), which is required in future parser (and thus current puppet) but perhaps doesn't work with the old parser. I'll see if it can be worked around.


@tvsonar First occurrence of an out-of-sync rule description :)


I think `len` should be of type `size_t`, as that is the type that `strlen` returns.


The Menu button is deprecated from Android 5 onwards and isn't built into new devices anymore.
Being abandoned in Lollipop, it's pretty much useless for us to have a Menu hwkey. An App-switch hwkey makes more sense here.


Hue Hue :p


```
>>> Lecture.query.filter(Lecture.documents.any(Document.id.in_([1, 2]))).all()
2015-06-15 13:32:23,616 INFO sqlalchemy.engine.base.Engine SELECT documents.lectures.id AS documents_lectures_id, documents.lectures.name AS documents_lectures_name, documents.l
ectures.aliases AS documents_lectures_aliases, documents.lectures.subject AS documents_lectures_subject, documents.lectures.comment AS documents_lectures_comment
FROM documents.lectures
WHERE EXISTS (SELECT 1
FROM documents.lecture_docs, documents.documents
WHERE documents.lectures.id = documents.lecture_docs.lecture_id AND documents.documents.id = documents.lecture_docs.document_id AND documents.documents.id IN (%(id_1)s, %(id_2)s
))
2015-06-15 13:32:23,616 INFO sqlalchemy.engine.base.Engine {'id_2': 2, 'id_1': 1}
[<models.documents.Lecture object at 0x7f6ba43decf8>, <models.documents.Lecture object at 0x7f6ba43deda0>, <models.documents.Lecture object at 0x7f6ba43dee10>, <models.documents
.Lecture object at 0x7f6ba43deeb8>, <models.documents.Lecture object at 0x7f6ba43def60>]
```


Hi Char,

Two new files were created under helloworld.

Cheers,
Mel


ここはforeachのほうが楽ですよ


Ten grid też wymaga przerobienia pod względem kolorów <- nie widać tekstu kiedy motyw jest jasny


would u tell me why? pls


maybe I just did not sync mine... :)


abc


@softwarenerd thanks for the commit. I was working on different scenarios and it looks like, addon (JXcore addon) may register itself.

 i.e. lets say jxcore has a [JXcore addNativeMethod:(callback) withName:(NSString)]

Addon searches for JXcore and sends a message to given above method.

The tricky part is, we don't know in what order Cordova will be registering the plugins. For example;
If the jxcore addon was first and jxcore itself second, there is no problem. We can register and use later.

However, if JXcore is already initialized before it's plugins and application code uses something specific to addon, there we have the problem. Firing an event on the application side may solve this problem. Trying to find if there is another way to solve the problem without bringing an additional event mechanism.


done.


I'll take a look at this.


It [defaults](https://getcomposer.org/doc/04-schema.md#minimum-stability) to stable.


ナイス！


why?


aaa


Just had a bit more of a look in to this and a nice approach would be:

```ruby
require 'delegate'

class ConvertedString < SimpleDelegator
  attr_reader :scrubbed

  def initialize(str, scrubbed)
    super(str)
    @scrubbed = scrubbed
  end

  alias_method :scrubbed?, :scrubbed
end

c = ConvertedString.new('abc', false)
# => "abc"
c
# => "abc"
c.scrubbed?
# => false
```

h/t http://words.steveklabnik.com/beware-subclassing-ruby-core-classes


EmptyPage not used.


OK, we will look at this and see if we merge it like this. Next time could you make it separate commits (marc fields added, change in the editor)? Thanks


Any way to do defer RemoveAll()?


@rassilon I had to add this or the `BrowserSubProcess` would crash when it disposed (just had to native away to a different page). Haven't looked any further to see if there is a more elegant solution.


:ok:


@mpaladin ahummm


I don't know lol... I was in the drug or something


you forget ```php
 global $PL;
```


Меня этот участок кода смущает

При `$sName` = `[test][new][here]` код `substr_replace(substr($sName, 0, -1), "]", $mPos, 0)` вернет `][test][new][here` и дальше `explode` разобьет его на массив с первым пустым элементом.


@mpaladin Aren't we supposed to have a non null by default annotation in this file?


lol wut , bro i created a new repo for downloading from, dont use releases repo, use fileserver repo (r/w should be available, test)


@diamondbond kk b8


1: M111 unrecognized component name: legacy


37: D103 Missing docstring in public function
47:13: E121 continuation line under-indented for hanging indent


1: D200 One-line docstring should fit on one line with quotes (found 3)
4: L102 copyright year is outdated, expected 2015 but got 2014
26:1: E402 module level import not at top of file
27:1: E402 module level import not at top of file
28:1: E402 module level import not at top of file
29:1: E402 module level import not at top of file
30:1: I100 Imports statements are in the wrong order. import calendar should be before import urlparse
30:1: E402 module level import not at top of file
31:1: I201 Missing newline before sections or imports.
31:1: E402 module level import not at top of file
32:1: E402 module level import not at top of file
34:1: I101 Imported names are in the wrong order. Should be CFG_SITE_ADMIN_EMAIL, CFG_SITE_URL
34:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.oaiharvest
34:1: E402 module level import not at top of file
37:1: I101 Imported names are in the wrong order. Should be field_xml_output, record_get_field_instances, record_modify_subfield
37:1: E402 module level import not at top of file
41:1: E402 module level import not at top of file
42:1: E402 module level import not at top of file
43:1: I100 Imports statements are in the wrong order. from invenio.legacy.oaiharvest.dblayer should be before from invenio.utils.text
43:1: E402 module level import not at top of file
44:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibcatalog.api should be before from invenio.legacy.oaiharvest.dblayer
44:1: F401 'BIBCATALOG_SYSTEM' imported but unused
44:1: E402 module level import not at top of file
46:1: I101 Imported names are in the wrong order. Should be task_low_level_submission, write_message
46:1: E402 module level import not at top of file
48:1: E402 module level import not at top of file
55: D400 First line should end with a period (not 'd')
55: D205 1 blank line required between summary line and description (found 0)
68: D400 First line should end with a period (not 'd')
68: D205 1 blank line required between summary line and description (found 0)
78: D400 First line should end with a period (not 's')
78: D205 1 blank line required between summary line and description (found 0)
91: D401 First line should be in imperative mood ('Collect', not 'Collects')
91: D209 Multi-line docstring closing quotes should be on a separate line
91: D400 First line should end with a period (not 't')
91: D205 1 blank line required between summary line and description (found 0)
110: D401 First line should be in imperative mood ('Thi', not 'This')
110: D400 First line should end with a period (not 'y')
110: D205 1 blank line required between summary line and description (found 0)
140: D400 First line should end with a period (not 'y')
140: D205 1 blank line required between summary line and description (found 0)
173: D400 First line should end with a period (not 'm')
173: D205 1 blank line required between summary line and description (found 0)
173: D208 Docstring is over-indented
203: D401 First line should be in imperative mood ('Return', not 'Returns')
203: D400 First line should end with a period (not 'o')
203: D205 1 blank line required between summary line and description (found 0)
227:5: B901 blind except: statement
290: D401 First line should be in imperative mood ('Perform', not 'Performs')
290: D205 1 blank line required between summary line and description (found 0)
326: D401 First line should be in imperative mood ('Create', not 'Creates')
326: D400 First line should end with a period (not 'e')
326: D205 1 blank line required between summary line and description (found 0)
398: D400 First line should end with a period (not 's')
398: D205 1 blank line required between summary line and description (found 0)
446: D401 First line should be in imperative mood ('Thi', not 'This')
446: D400 First line should end with a period (not 's')
446: D205 1 blank line required between summary line and description (found 0)
482: D401 First line should be in imperative mood ('Thi', not 'This')
499:8: F821 undefined name 'bibcatalog_system'
500:31: F821 undefined name 'bibcatalog_system'
507:16: F821 undefined name 'bibcatalog_system'
509:19: F821 undefined name 'bibcatalog_system'


4: L102 copyright year is outdated, expected 2015 but got 2014
427:25: E711 comparison to None should be 'if cond is None:'


1: M100 needs more reviewers
1: M101 signature is missing


30:1: I201 Missing newline before sections or imports.
31:1: I100 Imports statements are in the wrong order. import logging should be before from six
31:1: I201 Missing newline before sections or imports.
33:1: I201 Missing newline before sections or imports.
34:1: I101 Imported names are in the wrong order. Should be current_app, has_app_context
34:1: I100 Imports statements are in the wrong order. from flask should be before import six
34:1: I201 Missing newline before sections or imports.
35:1: I100 Imports statements are in the wrong order. from functools should be before from flask
35:1: I201 Missing newline before sections or imports.
35:1: F401 'wraps' imported but unused
36:1: I101 Imported names are in the wrong order. Should be find_modules, import_string
36:1: I201 Missing newline before sections or imports.
111:13: B901 blind except: statement
224:1: E302 expected 2 blank lines, found 1
228: D102 Missing docstring in public method
231: D102 Missing docstring in public method
239: D102 Missing docstring in public method
242: D102 Missing docstring in public method
246: D401 First line should be in imperative mood ('Run', not 'Runs')
298:23: E222 multiple spaces after operator


26:1: F401 'CFG_DEVEL_SITE' imported but unused
80: D102 Missing docstring in public method
109: D209 Multi-line docstring closing quotes should be on a separate line
109: D203 1 blank line required before class docstring (found 0)
109: D400 First line should end with a period (not 'f')
109: D205 1 blank line required between summary line and description (found 0)
115: D102 Missing docstring in public method
169: D102 Missing docstring in public method
177: D210 No whitespaces allowed surrounding docstring text
177: D209 Multi-line docstring closing quotes should be on a separate line
177: D203 1 blank line required before class docstring (found 0)
177: D400 First line should end with a period (not 'l')
177: D205 1 blank line required between summary line and description (found 0)
206: D102 Missing docstring in public method
231: D102 Missing docstring in public method


1: D100 Missing docstring in public module
28:1: F401 'sys' imported but unused
28:1: E402 module level import not at top of file
29:1: I100 Imports statements are in the wrong order. import datetime should be before import sys
29:1: E402 module level import not at top of file
30:1: E402 module level import not at top of file
31:1: I100 Imports statements are in the wrong order. import os should be before import time
31:1: E402 module level import not at top of file
33:1: E402 module level import not at top of file
34:1: I101 Imported names are in the wrong order. Should be CFG_CACHEDIR, CFG_INSPIRE_SITE, CFG_LOGDIR, CFG_TMPDIR, CFG_TMPDIR, CFG_TMPSHAREDDIR, CFG_WEBSEARCH_RSS_TTL, CFG_WEBSESSION_NOT_CONFIRMED_EMAIL_ADDRESS_EXPIRE_IN_DAYS
34:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.dbquery
34:1: F999 redefinition of unused 'CFG_TMPDIR' from line 34
34:1: F401 'CFG_LOGDIR' imported but unused
34:1: E402 module level import not at top of file
35:9: E126 continuation line over-indented for hanging indent
38:1: I101 Imported names are in the wrong order. Should be task_get_option, task_init, task_set_option, write_message, write_messages
38:1: E402 module level import not at top of file
39:10: E126 continuation line over-indented for hanging indent
40:1: E402 module level import not at top of file
41:1: E402 module level import not at top of file
42:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibdocfile.api should be before from invenio.modules.access.mailcookie
42:1: E402 module level import not at top of file
43:1: E402 module level import not at top of file
44:1: E402 module level import not at top of file
45:1: E402 module level import not at top of file
46:1: I100 Imports statements are in the wrong order. from intbitset should be before from invenio.utils.date
46:1: I201 Missing newline before sections or imports.
46:1: E402 module level import not at top of file
94:1: E302 expected 2 blank lines, found 1
94: D210 No whitespaces allowed surrounding docstring text
101:1: E302 expected 2 blank lines, found 1
101: D210 No whitespaces allowed surrounding docstring text
105:13: E128 continuation line under-indented for visual indent
108:9: E128 continuation line under-indented for visual indent
108:42: E502 the backslash is redundant between brackets
109:13: E128 continuation line under-indented for visual indent
111:9: E128 continuation line under-indented for visual indent
111:44: E502 the backslash is redundant between brackets
112:13: E128 continuation line under-indented for visual indent
114:9: E128 continuation line under-indented for visual indent
114:46: E502 the backslash is redundant between brackets
115:13: E128 continuation line under-indented for visual indent
118:1: E302 expected 2 blank lines, found 1
118: D210 No whitespaces allowed surrounding docstring text
122:13: E128 continuation line under-indented for visual indent
126:13: E128 continuation line under-indented for visual indent
128:9: E128 continuation line under-indented for visual indent
128:33: E502 the backslash is redundant between brackets
129:13: E128 continuation line under-indented for visual indent
131:9: E128 continuation line under-indented for visual indent
131:33: E502 the backslash is redundant between brackets
132:13: E128 continuation line under-indented for visual indent
135:9: E128 continuation line under-indented for visual indent
135:46: E502 the backslash is redundant between brackets
136:13: E128 continuation line under-indented for visual indent
136:46: E502 the backslash is redundant between brackets
139:9: E128 continuation line under-indented for visual indent
139:45: E502 the backslash is redundant between brackets
140:13: E128 continuation line under-indented for visual indent
140:46: E502 the backslash is redundant between brackets
144:13: E128 continuation line under-indented for visual indent
146:9: E128 continuation line under-indented for visual indent
146:44: E502 the backslash is redundant between brackets
147:13: E128 continuation line under-indented for visual indent
152:9: E128 continuation line under-indented for visual indent
152:44: E502 the backslash is redundant between brackets
153:13: E128 continuation line under-indented for visual indent
153:46: E502 the backslash is redundant between brackets
158:9: E128 continuation line under-indented for visual indent
158:44: E502 the backslash is redundant between brackets
159:13: E128 continuation line under-indented for visual indent
159:46: E502 the backslash is redundant between brackets
165:13: E128 continuation line under-indented for visual indent
165:48: E502 the backslash is redundant between brackets
166:17: E128 continuation line under-indented for visual indent
166:50: E502 the backslash is redundant between brackets
167:17: E128 continuation line under-indented for visual indent
171:13: E128 continuation line under-indented for visual indent
171:48: E502 the backslash is redundant between brackets
172:17: E128 continuation line under-indented for visual indent
172:50: E502 the backslash is redundant between brackets
173:17: E128 continuation line under-indented for visual indent
177:9: E128 continuation line under-indented for visual indent
177:44: E502 the backslash is redundant between brackets
178:13: E128 continuation line under-indented for visual indent
183:9: E128 continuation line under-indented for visual indent
183:44: E502 the backslash is redundant between brackets
184:13: E128 continuation line under-indented for visual indent
189:1: E302 expected 2 blank lines, found 1
233: D205 1 blank line required between summary line and description (found 0)
250:69: E502 the backslash is redundant between brackets
251:25: E128 continuation line under-indented for visual indent
256:69: E502 the backslash is redundant between brackets
257:25: E127 continuation line over-indented for visual indent
260:72: E502 the backslash is redundant between brackets
264:1: E302 expected 2 blank lines, found 1
264: D209 Multi-line docstring closing quotes should be on a separate line
264: D400 First line should end with a period (not 'n')
264: D205 1 blank line required between summary line and description (found 0)
280:1: E302 expected 2 blank lines, found 1
280: D400 First line should end with a period (not 'e')
280: D205 1 blank line required between summary line and description (found 0)
291:71: E261 at least two spaces before inline comment
293:1: E302 expected 2 blank lines, found 1
293: D400 First line should end with a period (not 'B')
293: D205 1 blank line required between summary line and description (found 0)
305:74: E261 at least two spaces before inline comment
307:1: E302 expected 2 blank lines, found 1
307: D401 First line should be in imperative mood ('Delete', not 'Deletes')
307: D200 One-line docstring should fit on one line with quotes (found 3)
322:1: E302 expected 2 blank lines, found 1
322: D401 First line should be in imperative mood ('Delete', not 'Deletes')
322: D400 First line should end with a period (not 's')
328:1: E302 expected 2 blank lines, found 1
328: D209 Multi-line docstring closing quotes should be on a separate line
328: D202 No blank lines allowed after function docstring (found 1)
328: D400 First line should end with a period (not 'r')
328: D208 Docstring is over-indented
360:34: E203 whitespace before ':'
368:9: E128 continuation line under-indented for visual indent
370:9: E128 continuation line under-indented for visual indent
373:5: E303 too many blank lines (2)
391:26: E701 multiple statements on one line (colon)
396:17: E128 continuation line under-indented for visual indent
398:17: E128 continuation line under-indented for visual indent
401:5: E303 too many blank lines (2)
405:9: E128 continuation line under-indented for visual indent
422:9: E128 continuation line under-indented for visual indent
445:5: E303 too many blank lines (2)
455:5: B901 blind except: statement
474:5: B901 blind except: statement
500:5: E266 too many leading '#' for block comment
526:1: E302 expected 2 blank lines, found 1
542:13: E128 continuation line under-indented for visual indent
543:13: E128 continuation line under-indented for visual indent
544:17: E127 continuation line over-indented for visual indent
555:13: E128 continuation line under-indented for visual indent
556:13: E128 continuation line under-indented for visual indent
557:13: E128 continuation line under-indented for visual indent
558:13: E128 continuation line under-indented for visual indent
559:13: E128 continuation line under-indented for visual indent
561:1: E302 expected 2 blank lines, found 1
561: D103 Missing docstring in public function
576:1: E302 expected 2 blank lines, found 1
576: D210 No whitespaces allowed surrounding docstring text
576: D400 First line should end with a period (not 'e')
576: D205 1 blank line required between summary line and description (found 0)
633:1: E302 expected 2 blank lines, found 1
633: D210 No whitespaces allowed surrounding docstring text


1: D100 Missing docstring in public module
21:1: E402 module level import not at top of file
22:1: I100 Imports statements are in the wrong order. import cgi should be before import urllib
22:1: E402 module level import not at top of file
24:1: E402 module level import not at top of file
25:1: I101 Imported names are in the wrong order. Should be CFG_ACCESS_CONTROL_LEVEL_ACCOUNTS, CFG_CERN_SITE, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_NAME_INTL, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL, CFG_WEBSEARCH_MAX_RECORDS_IN_GROUPS, CFG_WEBSESSION_ADDRESS_ACTIVATION_EXPIRE_IN_DAYS, CFG_WEBSESSION_RESET_PASSWORD_EXPIRE_IN_DAYS
25:1: F401 'CFG_ACCESS_CONTROL_LEVEL_ACCOUNTS' imported but unused
25:1: E402 module level import not at top of file
26:6: E126 continuation line over-indented for hanging indent
41:1: I101 Imported names are in the wrong order. Should be create_html_link, create_url, make_canonical_urlargd
41:1: E402 module level import not at top of file
42:1: I100 Imports statements are in the wrong order. from invenio.utils.html should be before from invenio.utils.url
42:1: F401 'nmtoken_from_string' imported but unused
42:1: E402 module level import not at top of file
43:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.utils.html
43:1: E402 module level import not at top of file
44:1: E402 module level import not at top of file
45:1: I100 Imports statements are in the wrong order. from invenio.legacy.websession.websession_config should be before from invenio.modules.apikeys.models
45:1: E402 module level import not at top of file
48: D101 Missing docstring in public class
70:17: E225 missing whitespace around operator
71:16: E126 continuation line over-indented for hanging indent
71:25: E203 whitespace before ':'
72:25: E203 whitespace before ':'
73:25: E203 whitespace before ':'
74:25: E203 whitespace before ':'
75:14: E126 continuation line over-indented for hanging indent
79: D102 Missing docstring in public method
80:9: F841 local variable '_' is assigned to but never used
88: D102 Missing docstring in public method
98:37: E203 whitespace before ':'
99:28: E203 whitespace before ':'
100:38: E203 whitespace before ':'
101:29: E203 whitespace before ':'
102:30: E203 whitespace before ':'
104:35: E203 whitespace before ':'
109:5: E303 too many blank lines (2)
109: D401 First line should be in imperative mood ('Display', not 'Displays')
109: D202 No blank lines allowed after function docstring (found 1)
109: D400 First line should end with a period (not 'r')
135:22: E126 continuation line over-indented for hanging indent
135:36: E203 whitespace before ':'
164:31: E126 continuation line over-indented for hanging indent
169:43: E203 whitespace before ':'
197:32: E203 whitespace before ':'
198:43: E203 whitespace before ':'
199:27: E203 whitespace before ':'
200:22: E203 whitespace before ':'
202:39: E203 whitespace before ':'
205:16: E126 continuation line over-indented for hanging indent
209: D401 First line should be in imperative mood ('Display', not 'Displays')
209: D202 No blank lines allowed after function docstring (found 1)
264:17: E126 continuation line over-indented for hanging indent
264:30: E203 whitespace before ':'
265:30: E203 whitespace before ':'
266:33: E203 whitespace before ':'
267:27: E203 whitespace before ':'
269:34: E203 whitespace before ':'
269:45: E225 missing whitespace around operator
270:34: E203 whitespace before ':'
270:45: E225 missing whitespace around operator
271:28: E203 whitespace before ':'
272:28: E203 whitespace before ':'
273:26: E203 whitespace before ':'
274:23: E203 whitespace before ':'
275:29: E203 whitespace before ':'
276:24: E203 whitespace before ':'
277:33: E203 whitespace before ':'
279:38: E203 whitespace before ':'
280:13: E121 continuation line under-indented for hanging indent
319:21: E126 continuation line over-indented for hanging indent
319:34: E203 whitespace before ':'
320:32: E203 whitespace before ':'
321:35: E203 whitespace before ':'
322:35: E203 whitespace before ':'
324:31: E203 whitespace before ':'
325:27: E203 whitespace before ':'
326:36: E203 whitespace before ':'
327:40: E203 whitespace before ':'
328:38: E203 whitespace before ':'
329:33: E203 whitespace before ':'
330:40: E203 whitespace before ':'
332:17: E121 continuation line under-indented for hanging indent
335:21: E128 continuation line under-indented for visual indent
335:34: E203 whitespace before ':'
337:139: E203 whitespace before ':'
338:35: E203 whitespace before ':'
341:30: E203 whitespace before ':'
341:96: E203 whitespace before ':'
345:5: E303 too many blank lines (2)
345: D400 First line should end with a period (not 'd')
363:11: E121 continuation line under-indented for hanging indent
363:26: E203 whitespace before ':'
364:32: E203 whitespace before ':'
365:32: E203 whitespace before ':'
366:37: E203 whitespace before ':'
367:21: E203 whitespace before ':'
368:21: E203 whitespace before ':'
369:28: E203 whitespace before ':'
375:5: E303 too many blank lines (2)
375: D102 Missing docstring in public method
383:11: E121 continuation line under-indented for hanging indent
383:26: E203 whitespace before ':'
384:31: E203 whitespace before ':'
388:27: E203 whitespace before ':'
389:27: E203 whitespace before ':'
390:26: E203 whitespace before ':'
395:26: E203 whitespace before ':'
396:30: E203 whitespace before ':'
402:5: E303 too many blank lines (2)
402: D102 Missing docstring in public method
410:11: E121 continuation line under-indented for hanging indent
410:26: E203 whitespace before ':'
411:26: E203 whitespace before ':'
414:23: E203 whitespace before ':'
415:19: E203 whitespace before ':'
418:23: E203 whitespace before ':'
419:19: E203 whitespace before ':'
424:30: E203 whitespace before ':'
430:5: E303 too many blank lines (2)
430:51: E251 unexpected spaces around keyword / parameter equals
430:53: E251 unexpected spaces around keyword / parameter equals
430:72: E251 unexpected spaces around keyword / parameter equals
430:74: E251 unexpected spaces around keyword / parameter equals
430:93: E251 unexpected spaces around keyword / parameter equals
430:95: E251 unexpected spaces around keyword / parameter equals
430: D102 Missing docstring in public method
442:11: E121 continuation line under-indented for hanging indent
442:26: E203 whitespace before ':'
443:36: E203 whitespace before ':'
444:27: E203 whitespace before ':'
445:30: E203 whitespace before ':'
446:25: E203 whitespace before ':'
447:28: E203 whitespace before ':'
453:25: E126 continuation line over-indented for hanging indent
453:35: E203 whitespace before ':'
454:28: E203 whitespace before ':'
455:21: E121 continuation line under-indented for hanging indent
461:17: E126 continuation line over-indented for hanging indent
461:34: E203 whitespace before ':'
462:39: E203 whitespace before ':'
464:13: E121 continuation line under-indented for hanging indent
467: D401 First line should be in imperative mood ('Display', not 'Displays')
467: D202 No blank lines allowed after function docstring (found 1)
480:256: E203 whitespace before ':'
480:280: E203 whitespace before ':'
480:302: E203 whitespace before ':'
502:20: E203 whitespace before ':'
503:19: E203 whitespace before ':'
504:11: E121 continuation line under-indented for hanging indent
508:45: E203 whitespace before ':'
509:46: E203 whitespace before ':'
510:45: E203 whitespace before ':'
512:46: E203 whitespace before ':'
516:15: E128 continuation line under-indented for visual indent
516:73: E201 whitespace after '{'
516:81: E203 whitespace before ':'
516:106: E202 whitespace before '}'
519:9: E303 too many blank lines (2)
521: D401 First line should be in imperative mood ('Display', not 'Displays')
521: D202 No blank lines allowed after function docstring (found 1)
521: D400 First line should end with a period (not 'n')
543:17: E126 continuation line over-indented for hanging indent
543:32: E203 whitespace before ':'
544:25: E128 continuation line under-indented for visual indent
545:15: E126 continuation line over-indented for hanging indent
553:22: E126 continuation line over-indented for hanging indent
553:26: E203 whitespace before ':'
554:37: E203 whitespace before ':'
555:38: E203 whitespace before ':'
556:20: E126 continuation line over-indented for hanging indent
561:11: E121 continuation line under-indented for hanging indent
561:15: E203 whitespace before ':'
562:26: E203 whitespace before ':'
563:27: E203 whitespace before ':'
569:9: E122 continuation line missing indentation or outdented
569:13: E203 whitespace before ':'
570:9: E122 continuation line missing indentation or outdented
570:23: E203 whitespace before ':'
571:9: E122 continuation line missing indentation or outdented
571:25: E203 whitespace before ':'
577:13: E122 continuation line missing indentation or outdented
577:17: E203 whitespace before ':'
578:13: E122 continuation line missing indentation or outdented
578:28: E203 whitespace before ':'
579:13: E122 continuation line missing indentation or outdented
579:31: E203 whitespace before ':'
584:11: E121 continuation line under-indented for hanging indent
584:15: E203 whitespace before ':'
585:24: E203 whitespace before ':'
586:27: E203 whitespace before ':'
593:15: E121 continuation line under-indented for hanging indent
593:27: E203 whitespace before ':'
594:30: E203 whitespace before ':'
595:13: E122 continuation line missing indentation or outdented
596:13: E122 continuation line missing indentation or outdented
605: D401 First line should be in imperative mood ('Display', not 'Displays')
605: D202 No blank lines allowed after function docstring (found 1)
605: D400 First line should end with a period (not 'e')
618:17: E225 missing whitespace around operator
620:19: E225 missing whitespace around operator
630: D401 First line should be in imperative mood ('Display', not 'Displays')
630: D202 No blank lines allowed after function docstring (found 1)
630: D400 First line should end with a period (not 'r')
653: D401 First line should be in imperative mood ('Display', not 'Displays')
653: D202 No blank lines allowed after function docstring (found 1)
653: D400 First line should end with a period (not 'e')
668:14: E225 missing whitespace around operator
679: D401 First line should be in imperative mood ('Display', not 'Displays')
679: D202 No blank lines allowed after function docstring (found 1)
679: D400 First line should end with a period (not 'e')
741:32: E128 continuation line under-indented for visual indent
747:36: E128 continuation line under-indented for visual indent
751:35: E222 multiple spaces after operator
752:36: E222 multiple spaces after operator
753:35: E128 continuation line under-indented for visual indent
758:33: E222 multiple spaces after operator
759:34: E222 multiple spaces after operator
766: D401 First line should be in imperative mood ('Display', not 'Displays')
766: D202 No blank lines allowed after function docstring (found 1)
766: D400 First line should end with a period (not 'd')
780:14: E225 missing whitespace around operator
781:15: E225 missing whitespace around operator
788:17: E203 whitespace before ':'
789:18: E203 whitespace before ':'
790:24: E203 whitespace before ':'
791:11: E121 continuation line under-indented for hanging indent
794: D202 No blank lines allowed after function docstring (found 1)
794: D400 First line should end with a period (not 't')
794: D205 1 blank line required between summary line and description (found 0)
813:17: E128 continuation line under-indented for visual indent
814:17: E128 continuation line under-indented for visual indent
815:33: E203 whitespace before ':'
815:34: E231 missing whitespace after ':'
816:30: E203 whitespace before ':'
817:35: E203 whitespace before ':'
820:21: E203 whitespace before ':'
821:19: E203 whitespace before ':'
822:17: E131 continuation line unaligned for hanging indent
823:25: E203 whitespace before ':'
824:24: E203 whitespace before ':'
826:20: E203 whitespace before ':'
827:21: E203 whitespace before ':'
831: D202 No blank lines allowed after function docstring (found 1)
831: D400 First line should end with a period (not 'e')
831: D205 1 blank line required between summary line and description (found 0)
850:17: E128 continuation line under-indented for visual indent
851:17: E128 continuation line under-indented for visual indent
852:33: E203 whitespace before ':'
852:34: E231 missing whitespace after ':'
853:30: E203 whitespace before ':'
854:35: E203 whitespace before ':'
857:21: E203 whitespace before ':'
858:19: E203 whitespace before ':'
859:17: E131 continuation line unaligned for hanging indent
860:25: E203 whitespace before ':'
861:33: E203 whitespace before ':'
863:20: E203 whitespace before ':'
864:21: E203 whitespace before ':'
868: D401 First line should be in imperative mood ('Display', not 'Displays')
868: D202 No blank lines allowed after function docstring (found 1)
868: D400 First line should end with a period (not 't')
882:14: E225 missing whitespace around operator
886: D401 First line should be in imperative mood ('Display', not 'Displays')
886: D202 No blank lines allowed after function docstring (found 1)
886: D400 First line should end with a period (not 't')
901: D401 First line should be in imperative mood ('Display', not 'Displays')
901: D202 No blank lines allowed after function docstring (found 1)
901: D400 First line should end with a period (not 't')
901: D205 1 blank line required between summary line and description (found 0)
906:9: F841 local variable '_' is assigned to but never used
910:22: E126 continuation line over-indented for hanging indent
910:26: E203 whitespace before ':'
915: D401 First line should be in imperative mood ('Display', not 'Displays')
915: D202 No blank lines allowed after function docstring (found 1)
915: D400 First line should end with a period (not 'r')
997: D401 First line should be in imperative mood ('Display', not 'Displays')
997: D202 No blank lines allowed after function docstring (found 1)
997: D400 First line should end with a period (not 'k')
1043:22: E126 continuation line over-indented for hanging indent
1044:26: E203 whitespace before ':'
1045:33: E203 whitespace before ':'
1046:31: E203 whitespace before ':'
1047:29: E203 whitespace before ':'
1048:20: E126 continuation line over-indented for hanging indent
1052:21: E126 continuation line over-indented for hanging indent
1052:36: E203 whitespace before ':'
1053:25: E203 whitespace before ':'
1054:31: E203 whitespace before ':'
1055:16: E121 continuation line under-indented for hanging indent
1057:21: E126 continuation line over-indented for hanging indent
1057:36: E203 whitespace before ':'
1058:25: E203 whitespace before ':'
1059:29: E203 whitespace before ':'
1060:17: E121 continuation line under-indented for hanging indent
1063: D400 First line should end with a period (not 's')
1063: D205 1 blank line required between summary line and description (found 0)
1084: D400 First line should end with a period (not 'r')
1084: D205 1 blank line required between summary line and description (found 0)
1091:9: F841 local variable '_' is assigned to but never used
1100: D401 First line should be in imperative mood ('Display', not 'Displays')
1100: D202 No blank lines allowed after function docstring (found 1)
1100: D400 First line should end with a period (not 'l')
1104:52: E251 unexpected spaces around keyword / parameter equals
1104:54: E251 unexpected spaces around keyword / parameter equals
1125:9: F841 local variable '_' is assigned to but never used
1140:14: E225 missing whitespace around operator
1150:14: E225 missing whitespace around operator
1154:5: E303 too many blank lines (2)
1154: D202 No blank lines allowed after function docstring (found 1)
1173:9: E303 too many blank lines (2)
1174:102: E202 whitespace before ')'
1194:35: E225 missing whitespace around operator
1201:15: E225 missing whitespace around operator
1204:46: E203 whitespace before ':'
1205:44: E203 whitespace before ':'
1207:42: E231 missing whitespace after ':'
1208:43: E231 missing whitespace after ':'
1209:45: E231 missing whitespace after ':'
1211:49: E203 whitespace before ':'
1212:47: E203 whitespace before ':'
1214:45: E231 missing whitespace after ':'
1215:46: E231 missing whitespace after ':'
1216:48: E231 missing whitespace after ':'
1275:15: E225 missing whitespace around operator
1299:16: E127 continuation line over-indented for visual indent
1300:29: E231 missing whitespace after ':'
1301:16: E124 closing bracket does not match visual indentation
1305:5: E303 too many blank lines (2)
1338:15: E225 missing whitespace around operator
1352: D400 First line should end with a period (not ':')
1352: D205 1 blank line required between summary line and description (found 0)
1375:20: E225 missing whitespace around operator
1390:46: E231 missing whitespace after ','
1459:25: E203 whitespace before ':'
1462:28: E203 whitespace before ':'
1470:30: E231 missing whitespace after ':'
1471:31: E231 missing whitespace after ':'
1472:28: E231 missing whitespace after ':'
1474:30: E203 whitespace before ':'
1474:31: E231 missing whitespace after ':'
1477:16: E124 closing bracket does not match visual indentation
1480: D201 No blank lines allowed before function docstring (found 1)
1480: D205 1 blank line required between summary line and description (found 0)
1571:25: E203 whitespace before ':'
1575:25: E231 missing whitespace after ':'
1576:29: E231 missing whitespace after ':'
1579:31: E231 missing whitespace after ':'
1580:29: E203 whitespace before ':'
1580:30: E231 missing whitespace after ':'
1580:64: E231 missing whitespace after ','
1581:33: E203 whitespace before ':'
1582:16: E124 closing bracket does not match visual indentation
1585: D202 No blank lines allowed after function docstring (found 1)
1720:19: E203 whitespace before ':'
1721:26: E222 multiple spaces after operator
1726:37: E231 missing whitespace after ','
1726:23: E222 multiple spaces after operator
1727:13: E203 whitespace before ':'
1729:27: E203 whitespace before ':'
1730:27: E222 multiple spaces after operator
1738:38: E231 missing whitespace after ','
1738:23: E222 multiple spaces after operator
1739:13: E203 whitespace before ':'
1748:21: E203 whitespace before ':'
1749:30: E203 whitespace before ':'
1750:27: E203 whitespace before ':'
1758:9: E122 continuation line missing indentation or outdented
1758:73: E201 whitespace after '{'
1758:81: E203 whitespace before ':'
1759:65: E128 continuation line under-indented for visual indent
1759:78: E203 whitespace before ':'
1760:65: E124 closing bracket does not match visual indentation
1761:26: E128 continuation line under-indented for visual indent
1765:13: E128 continuation line under-indented for visual indent
1767:24: E231 missing whitespace after ':'
1768:30: E203 whitespace before ':'
1769:31: E203 whitespace before ':'
1769:32: E231 missing whitespace after ':'
1770:25: E231 missing whitespace after ':'
1771:24: E231 missing whitespace after ':'
1780:31: E231 missing whitespace after ':'
1781:21: E231 missing whitespace after ':'
1849:21: E222 multiple spaces after operator
1852:13: E203 whitespace before ':'
1858:25: E203 whitespace before ':'
1859:29: E203 whitespace before ':'
1860:25: E231 missing whitespace after ':'
1862:24: E203 whitespace before ':'
1863:31: E231 missing whitespace after ':'
1864:21: E203 whitespace before ':'
1864:22: E231 missing whitespace after ':'
1865:25: E203 whitespace before ':'
1870:5: E303 too many blank lines (2)
1870: D400 First line should end with a period (not 'p')
1870: D205 1 blank line required between summary line and description (found 0)
1901:11: E225 missing whitespace around operator
1902:19: E231 missing whitespace after ':'
1905:22: E231 missing whitespace after ':'
1907:15: E124 closing bracket does not match visual indentation
1910: D400 First line should end with a period (not 'e')
1910: D205 1 blank line required between summary line and description (found 0)
1941:11: E225 missing whitespace around operator
1942:19: E231 missing whitespace after ':'
1945:22: E231 missing whitespace after ':'
1947:15: E124 closing bracket does not match visual indentation
1950: D400 First line should end with a period (not 'y')
1950: D205 1 blank line required between summary line and description (found 0)
1964: D210 No whitespaces allowed surrounding docstring text
1964: D400 First line should end with a period (not 'u')
1964: D205 1 blank line required between summary line and description (found 0)
1969:20: E203 whitespace before ':'
1971:62: E225 missing whitespace around operator
1972:13: E203 whitespace before ':'
1979:54: E225 missing whitespace around operator
1984:5: E303 too many blank lines (2)
1984: D400 First line should end with a period (not 's')
1984: D205 1 blank line required between summary line and description (found 0)
1990:9: F841 local variable '_' is assigned to but never used
2002: D400 First line should end with a period (not ':')
2002: D205 1 blank line required between summary line and description (found 0)
2017:15: E271 multiple spaces after keyword
2019: D400 First line should end with a period (not ':')
2019: D205 1 blank line required between summary line and description (found 0)
2036: D400 First line should end with a period (not 'p')
2036: D205 1 blank line required between summary line and description (found 0)
2055: D400 First line should end with a period (not 'd')
2055: D205 1 blank line required between summary line and description (found 0)
2080: D400 First line should end with a period (not 'd')
2080: D205 1 blank line required between summary line and description (found 0)
2098: D400 First line should end with a period (not ')')
2098: D205 1 blank line required between summary line and description (found 0)
2116: D400 First line should end with a period (not 'e')
2116: D205 1 blank line required between summary line and description (found 0)
2164:19: E128 continuation line under-indented for visual indent


1: D210 No whitespaces allowed surrounding docstring text
27:1: I101 Imported names are in the wrong order. Should be CFG_SITE_ADMIN_EMAIL, CFG_SITE_NAME, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL
27:1: E402 module level import not at top of file
28:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.config
28:1: E402 module level import not at top of file
29:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.base.i18n
29:1: F401 'config' imported but unused
29:1: E402 module level import not at top of file
146:16: E126 continuation line over-indented for hanging indent
202:15: E126 continuation line over-indented for hanging indent
205:1: E402 module level import not at top of file
238:17: E203 whitespace before ':'
239:20: E203 whitespace before ':'
240:18: E203 whitespace before ':'
241:22: E203 whitespace before ':'
242:19: E203 whitespace before ':'
243:20: E203 whitespace before ':'
244:23: E203 whitespace before ':'
245:18: E203 whitespace before ':'
246:17: E203 whitespace before ':'
247:19: E203 whitespace before ':'
248:20: E203 whitespace before ':'
249:19: E203 whitespace before ':'
250:20: E203 whitespace before ':'
251:17: E203 whitespace before ':'
252:24: E203 whitespace before ':'
253:21: E203 whitespace before ':'
254:37: E203 whitespace before ':'
268:4: E131 continuation line unaligned for hanging indent
297:1: W391 blank line at end of file


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: collections


28:1: I201 Missing newline before sections or imports.
29:1: I100 Imports statements are in the wrong order. from invenio.modules.records.utils should be before from invenio_records.signals
29:1: I201 Missing newline before sections or imports.


1: D100 Missing docstring in public module
27: D103 Missing docstring in public function


44:32: E711 comparison to None should be 'if cond is not None:'
67: D103 Missing docstring in public function


1: D100 Missing docstring in public module


1: M100 needs more reviewers
1: M101 signature is missing
1: M110 missing component name


23:1: I100 Imports statements are in the wrong order. from invenio.modules.search.api should be before from invenio_records.signals
23:1: I201 Missing newline before sections or imports.
44:32: E711 comparison to None should be 'if cond is not None:'
67: D103 Missing docstring in public function


24:1: I100 Imports statements are in the wrong order. from flask should be before import six
25:1: I201 Missing newline before sections or imports.
26:1: I201 Missing newline before sections or imports.
27:1: I101 Imported names are in the wrong order. Should be MultipleResultsFound, NoResultFound
28:1: I201 Missing newline before sections or imports.
30:1: I100 Imports statements are in the wrong order. from invenio.base.utils should be before from werkzeug.utils
31:1: I100 Imports statements are in the wrong order. from invenio.base.helpers should be before from invenio.base.utils
34:1: I101 Imported names are in the wrong order. Should be SmartDict, SmartJson
35:1: I100 Imports statements are in the wrong order. from invenio.modules.jsonalchemy.errors should be before from invenio.modules.jsonalchemy.wrappers
41: D101 Missing docstring in public class
44: D102 Missing docstring in public method
71: D102 Missing docstring in public method
74: D102 Missing docstring in public method
80: D103 Missing docstring in public function
255:33: W503 line break before binary operator
256:33: W503 line break before binary operator


23:1: I201 Missing newline before sections or imports.


1: D100 Missing docstring in public module
37: D103 Missing docstring in public function


24:1: I100 Imports statements are in the wrong order. from flask should be before import six
25:1: I201 Missing newline before sections or imports.
26:1: I101 Imported names are in the wrong order. Should be MultipleResultsFound, NoResultFound
27:1: I201 Missing newline before sections or imports.
29:1: I100 Imports statements are in the wrong order. from invenio.modules.jsonalchemy.reader should be before from werkzeug.utils
31:1: I100 Imports statements are in the wrong order. from invenio.modules.jsonalchemy.errors should be before from invenio.modules.jsonalchemy.wrappers
203:33: W503 line break before binary operator
204:33: W503 line break before binary operator


1: D100 Missing docstring in public module
64: D103 Missing docstring in public function


1: D100 Missing docstring in public module
23:1: I201 Missing newline before sections or imports.
37: D103 Missing docstring in public function


1: D100 Missing docstring in public module
23:1: I201 Missing newline before sections or imports.
64: D103 Missing docstring in public function


b8? ;-;


b**ch


OK


Ahhh lol ok


third param is never used now


will you use OOP in the core? i liked the project


`make_shared()` одобряю


Added a comment in 90a52ac8f438d378e1a330ebc41df366d162bee5


`each` will not work because the purpose of the function is to delete all items in the array.  For example:

```
a = [1,2,3,4,5]

a.each do |element|
  a.delete(element)
end

=> [2, 4]
```


I like it, great m8 @Diamondbond


ИМХО
```
const std::shared_ptr<Bin_Tree_Element> up(bool update_counter = true) const
{
    return u.lock();
}
```
Так проще пользоваться снаружи — не нужно постоянно писать `up().lock()`.
Да и раз не поменяется интерфейс, вообще не нужно менять ничего в местах использования.
И для неконстантного `up()` аналогично.
Это самое существенное замечание.
Хороший продуманный интерфейс — главное.


Тут как раз видно что лучше возвращать из `up()` `shared_ptr`, а не `weak_ptr`.
Не пришлось бы писать столько раз `lock()`.
Да и вообще, в этом файле было бы меньше изменений.


`name` в таком формате можно указывать? Код писался из расчета что `name` указывается в формате `test[new][here]`


@dbolkensteyn already fixed in https://github.com/SonarCommunity/sonar-resharper/commit/b41145d9f05fe6ad998059548de9892feff8dbbc.


А зачем тебе это файл (`bin_tree.cpp.back`)?
У тебя же все старые версии есть в системе контроля версий?
Или он отличался? Тогда можно делать stash или временную ветку для экспериментов.


He had a good run


where does it unzip the file to? do you need an argument to define the output file? if not, then at least at a return value of the directory the file was unzipped to. Avoid ``void`` methods as much as you can :-)


I think it's time for another coffee before I break something!


What about a unit test for this?


cache return value


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Shouldn't this be managed upstream?


точно, все верно, я ошибся


Should be `_MSC_VER < 1400`


@laruence Could you change the 'yes' in 'no' in both branches? I cloned msgpack besides the other extensions in /ext. With the 'yes' a snapshot build tries to compile it as static, with the 'no' it tries to compile it as shared. It still fails in PHP7 but I guess you are working on that.


Do we want the original hostname/`HostKey` behaviour anywhere?


@eidheim search now works conceptually. Minor work left.


I looked through it just because I stumbled on this, looks good. That said, I don't know enough to validate correctness.


Ok, sure. Thanks!

2015-06-15 13:49 GMT+02:00 Laurent Pugin <notifications@github.com>:

> OK, we will look at this and see if we merge it like this. Next time could
> you make it separate commits (marc fields added, change in the editor)?
> Thanks
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rism-ch/muscat/commit/ecbf52048990c319a21abc39957f017a7afc4908#commitcomment-11678462>
> .
>

--

Répertoire International des Sources Musicales
Stephan Hirsch
IT-Manager
Zentralredaktion
Sophienstraße 26
D-60487 Frankfurt am Main
Tel: +49 (0) 69 70 62 31
Fax: +49 (0) 69 70 60 26
http://www.rism.info


This commit closes #4


Trocar a Referência Ramaswami2009 por Ramaswami2010


ひとまずタンヤオ、清老頭、混老頭あたりじゃないですか？？
書けるのかどうかわからんけど、
ifでstring(なのか？)1,9,ton,nan,sha,pe,haku,hatsu,chunのいずれかが入ってなかったらタンヤオみたいなさ。
その流れで清老頭、混老頭はタンヤオの２３４５６７８が入ってたらダメか、上の条件に合致する項目のみとか。

ここでみると、国士無双、字一色とか大四喜(なおアタマが１，９)とかいう問題が浮上してくるのでまた考えないといけないんですけど。
役満が変動しない(しにくい？)というか決まった点数、限定された手の内なので、そちらを実装してみては～～って話かも？


Можно упростить условие по закону [де Моргана] (https://ru.wikipedia.org/wiki/%C7%E0%EA%EE%ED%FB_%E4%E5_%CC%EE%F0%E3%E0%ED%E0):
```
if ((element->right() and element->right()->colour() != node_colour::black) or
    (element->left() and element->left()->colour() != node_colour::black))
```


```
if (way_black != cur_way)
```


Usually the name for boolean variables do not use the verb "is", i.e. hidden instead of isHidden. The corresponding getter method use "is" to answer the question whether the value of the variable is true or false.


$val should contribute to  "exit $val" in the end of script to indicate error.


@mattburton I know it's been a while :) but do you remember why we did this change?


Done now the error response will contain, an error code, error detail (which is having aadhaar number) and the http status, required some more polishing on http status code and will do it once after real implementation with uidai server.


Good catch. Thanks.


Hey you I tested out your Bot and I belive that you have to give some dangers priority to others.
If the bot gets close to an object it has the same fear as i gets in range of another object which is able to *shoot" it. I think with this improovment the bot will be able to do alot better results.

Another part is that the bot isnt able to split itself. This is maybe hard to programm...
With this feature the bot will be a lot faster at high results.

And if you split the bot it only looks at its main object and does not care about its other parts.

This should be constructive critic. But till now you did a great job. :+1:


#1 improved


I'm not sure I'm following


I realise this is just swapping out with old behaviour, but I think it might generally be preferable to avoid having a blanket redirect based on the existence of particular parameters as this might have unintended consequences (e.g. someone adds a call which takes a vm_uuid and a vdi_uuid - the current logic wouldn't be able to determine which object is of interest, and hence which host to execute the call against).


Старый вариант можно сразу удалять, он же сохраняется в системе контроля версий.


I hadn't actually tested the deployment recommended in the readme until now, nor the minified files, however I've made a separate deployment that is working to confirm everything works before writing a wiki for this.


This commit kills https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/virtualisation/ec2-data.nix#L36-L69. I don't think those are sane preferences, and that we should pull them out of base.


pathSegment could be named variables no? it it mandatory here to name the pathparam? ("path")


do we need these #ifdefs now that OS X is C++11 only?


(Well, this commit and https://github.com/zalora/microgram/commit/20ab8d53a7c195add60ae604726201fec3e9fa1d).


@dbolkensteyn fixed in https://github.com/SonarCommunity/sonar-resharper/commit/fc9f75c1764915431591be85065e2cd18617b095.


You could extend the object model in `api_helper.py` if you wanted - but you may have considered that ;-)


[knit] Is it on purpose that this is a '##' title, while Actions & Attributes are '###' title ?


FileUtils.deleteQuietly can be used here too (since we dont check the delete return)


error message could explain what we were trying to do like "Could save temporary file when submitting"?


I'd imagine it might be helpful to be able to trace how the decision for which host to executed against was determined would be helpful - particularly when going through logs?


Uhm.. Why all this? It looks quite horrible...

How is this any better than using data attributes to trigger functionality of ui_components?


here you can probably use an utility method from commons-lang or guava


can it be null?


**Note:** The parameter from the JCL will be put into register 1.

```jcl
EXEC PGM=MYPROG,PARAM='SOME STRING'
```

R1 will point to a location which has a length (2 bytes) and the data in the parameter value.


Can you use https://docs.djangoproject.com/en/1.8/ref/validators/#module-django.core.validators instead of this?

I know we have an older version of Django which may be before validators was implemented.


Good explanation about setting up the development enviroment for the ownCloud-Android app.


I got you know. Makes sense. I'm surprised we didn't kill this earlier.


General question: is the `https://localhost/export_raw_vdi` case the only case under consideration here?

If so, I wonder whether it would just be simpler to replace `localhost` with the host IP of a host with a PBD connected to the SR (rather than running the plugin call on a different host?).

There may well be a reason why you've decided that's not a good idea.


part1 => inputPart? and fileInputPart above?


update layer set layer_order=sqrt(power(layer_order-1000,2)) ;
update style set style_order=sqrt(power(style_order-10,2)) ;


X


The logic that returns a region based on the Department's state should be a method on the FireDepartment model.  The migration should just execute the method for each record.


looks like this class could download anything no?


Because the Markdown is part of the modal we load asynchronously, way after the DOM ready event. Do you have a better idea?


**Note:** The parameter has a length of 2 bytes which prefixes the value, so the point to the start of the data we must displace the parameter by 2 bytes.


Hi @donsavoie,

Where did you try to upload this plug-in as a zip file?

As it says in the [readme], this is a Javascript plug-in to change look of the Google Translate plugin after you have it added to your website.

If you have any question don't hesitate to ask, thanks!

[readme]: https://github.com/wistcc/stylinggt.js/blob/master/README.md


that would be awsome to change this api to be able to send it a stream instead of a file than we won' have to write this temporary file


ok


`@param childName` ;-)


On my an old Huawei U9500, this button works fine.
Why, then setting https://github.com/TeamEOS/frameworks_base/blob/d13693cc12506e5544f13a6e10093ec082e99da2/core/res/res/values/config.xml#L900 ?
The only thing that was able, to do so:
https://github.com/ShevT/device_samsung_lt033g/blob/cm-12.1/overlay/frameworks/base/core/res/res/values/config.xml#L257
The button works, but in the system settings is not possible to reassign


oh really i dont think so


shit piece of code


not sure this will fail the test


then you can use expected parameter in @Test


- variables updated.
- h tag styling removed (class styling will be moved into fonts.less).


Thank you. I noticed that this file needs tinkering when using Python 2.7 instead of 2.6.

I'm afraid your version requirements are unnecessarily strict. For instance, lxml>=3.3.1 is working fine. Why have you changed this line?


TeamCity NEventStore :: MongoDB :: Build [Build 21](http://build.prxm.it:8811/viewLog.html?buildId=3022&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 75 Build time: 00:00:39


could you please use all vg params as variable and use it everywhere as:

``` shell
valgrind $vg_opt pmix_test ...
```

Also, please use following vg params to generate xml report (jenkins will pick it up and render in UI and also can keep track of it)

``` shell
VALGRIND_ARGS = \
   --xml=yes --xml-file=valgrind.xml \
    --tool=memcheck \
    --leak-check=full \
    --track-origins=yes \
    --fair-sched=try \
    --gen-suppressions=all
```


is the headerparam case insensitive?


it is


Yes, we should use javascript/jquery events properly.

Something like this: http://stackoverflow.com/a/23324764/1252065


I added a few inline comments


thx


Also, what happened to pull requests? Introducting global state in ui_components which is used in many projects without any review process is usually a very bad idea...


@benbc  I've confirmed that that syntax won't work.  The best I can do is;

```
Powershell -NoProfile -ExecutionPolicy Bypass ^
 -Command "$ErrorActionPreference = 'Stop'; Import-Module '%~dp0Neo4j-Management.psd1'; Exit (Get-Neo4jServer '%~dp0..' | Start-Neo4jServer -Console -Wait)"
```

I can't split the powershell command with a caret as cmd.exe still splits the string.


TeamCity NEventStore :: MongoDB :: Build [Build 22](http://build.prxm.it:8811/viewLog.html?buildId=3023&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 75 Build time: 00:03:19


Is there some way I can get rid of these variables?

`cljr--occurrences` is a list of all the occurrences for the symbol.
`cljr--signature-changes` is a list of changes to the signature.  For each parameter, what's the old and new index and old and new name.

The reason I've put them out here is because the occurrences have to be collected before I display the interaction buffer and the signature changes are updated while the user interacts with said buffer.


Ouais, les "mountpoint for X not found" c'est un bug sur Docker.
apt-get install cgroup (or something)
`cat /proc/self/mountinfo` doit au moins contenir
```
25 21 0:18 / /cgroup/cpuset rw,relatime - cgroup cgroup rw,cpuset
26 21 0:19 / /cgroup/cpu rw,relatime - cgroup cgroup rw,cpu
27 21 0:20 / /cgroup/cpuacct rw,relatime - cgroup cgroup rw,cpuacct
28 21 0:21 / /cgroup/memory rw,relatime - cgroup cgroup rw,memory
29 21 0:22 / /cgroup/devices rw,relatime - cgroup cgroup rw,devices
30 21 0:23 / /cgroup/freezer rw,relatime - cgroup cgroup rw,freezer
31 21 0:24 / /cgroup/net_cls rw,relatime - cgroup cgroup rw,net_cls
32 21 0:25 / /cgroup/blkio rw,relatime - cgroup cgroup rw,blkio
```


Creating a new major-mode for the user interaction buffers seems a little overkill.  All I really need is a buffer with a few custom keybindings.


Triggering an event or calling a function doesn't really make a difference in terms of prettier code in this case. What do you mean with "global state"?

Sorry about the lack of a pull-request.


J'ai essayé , ca marche mais j'arrive pas à faire rentrer les modifs dans github ...


Thanks! BitHub has sent payment of  $8.30USD for this commit.


This module deserves to die.


hm... magit does something similar i think...


Kubernetes renamed portalIP to clusterIP, this change tracks that change.
No point in checking and bailing if the nginx config file is broken as nginx will itself bail if it is.


Yeah, you should probably deprecate it.


LGTM


ALERT


LGTM (though maybe it'd be more obvious with `token->string->length + 1 - powershellScopeLen - 1`?  whichever you prefer though, I don't think it matters much)


Hi, I think
line 114: patchHttpServer should be changed to "server = origHttpCreateServer.apply(__http__, arguments)"


I need a software to key generator passwords to SS02 ICs , who has this software to sell? Any information send email to katieneferreira@gmail.com


Why would that not be prettier?
- no inline javascript
- no random global DOM IDs
- no need to expose ui_components javascript functions to the app

Exposing the markdown render function to an object in the global namespace, which is only used internally in ui_components is what I meant by "global state". That's usually bad practise.


+1 - This issue threw me as well


@bchavez :

   Today we are getting strange issue, even with Live implementation for Coinbase, with iFrame binding in our website payment page. Instead of displaying the bitcoin address on which bitcoin payment accepted, it is displaying below message:

  X This content cannot be displayed in a frame
To help protect the security of information you enter into this website, the publisher of this content does not allow it to be displayed in a frame.

What you can try:
Open this content in a new window

 Attached the Actual and Strange behaviors of coinbase iFrame recently.
![coinbase actual iframe](https://cloud.githubusercontent.com/assets/2277305/8160131/bb9c8b10-138a-11e5-9f83-afb015cdb03b.png)
![coinbase iframe](https://cloud.githubusercontent.com/assets/2277305/8160132/bba31548-138a-11e5-8b6b-446eb8942ebe.png)

 Please guide us to get over this behavior. Thanks in advance.


@Guria это абстрактный список, который содержит просто набор известных шаблонизаторов


Thanks, thats a very good thing to happen :)


@enix instead of cellMetrics.metrics.removeEntity(routedCellMetrics.routerEntity) should not be routedCellMetrics.metrics.removeEntity(routedCellMetrics.routerEntity)?


[heavywater] New hw-redis created (version: v0.1)

- http://labs.hw-ops.com/jobs/14552ff4-5a43-4d9a-87cc-00004fc70000


OH yeah, this is probably one of the only lists we should have actually kept a directive as it was being uesd on the project pages as well...


AppSwitch key should be configurable for single tap, double tap, and long press actions, including the menu action. Double check Button settings


not needed


fix INSTANCEOF


@Parbhat here's what we're doing in the templates with the images for Facebook feed
````
    {% image self.feed_image width-1200 as img %}
    <meta property="og:image" content="http://birdsongtt.org{{ img.url }}" />
````
Have a look at the code for the `wagtailmagetags`


Here is how I used VoyagerOne 10 game data.
It will be interesting to have data for much more games,
or use his formula during the game to gather cutoff statistics and dynamically
find the best cut off expectations.

So I just sorted by VV and cut (not by cut%), and gave V=2 for B and N.

A	V	VV	cut	noCut	Total	cut %	Rank when sorted by VV and cut
K	P	1	241704	2502071	2743775	0.088092	1
Q	P	1	319996	1719857	2039853	0.156872	2
R	P	1	485470	3783116	4268586	0.113731	3
N	P	1	560650	2330986	2891636	0.193887	4
B	P	1	567295	3383750	3951045	0.143581	5
P	P	1	1286307	2170805	3457112	0.372076	6
K	N	2	20597	354505	375102	0.05491	7
B	B	2	35361	601474	636835	0.055526	8
K	B	2	38184	892601	930785	0.041023	9
N	N	2	64697	404766	469463	0.137811	10
N	B	2	65774	704035	769809	0.085442	11
Q	N	2	71683	502363	574046	0.124873	12
R	N	2	88267	626579	714846	0.123477	13
Q	B	2	93565	846448	940013	0.099536	14
R	B	2	108961	847208	956169	0.113956	15
P	B	2	133849	952825	1086674	0.123173	16
B	N	2	179118	761761	940879	0.190373	17
P	N	2	313315	821879	1135194	0.276001	18
K	R	3	17270	924123	941393	0.018345	19
Q	R	3	25865	257719	283584	0.091208	20
N	R	3	29313	322587	351900	0.083299	21
P	R	3	34521	399773	434294	0.079488	22
B	R	3	42909	433564	476473	0.090055	23
R	R	3	63752	271558	335310	0.190129	24
K	Q	4	02598	90521	93119	0.0279	25
Q	Q	4	02786	00478	003264	0.853554	26
N	Q	4	24617	182486	207103	0.118864	27
B	Q	4	32291	294005	326296	0.098962	28
P	Q	4	39251	235515	274766	0.142852	29
R	Q	4	39888	222183	262071	0.152203	30


Thanks :)


Pull in reworked activity markers via bpmn-io/bpmn-font#5


anyrefmap


Thank you


This change still doesn't cover all situations:
Eg: Imagine an school from 1900 to 1930 with name changes in 1910 and 1920 (3 names in total)
Suppose there was a principal from 1909 to 1922
The first inBetween returns 2nd and 3rd name
The second one returns 1st name


Oh... but this is inside a filter function... Never mind! it is good!


Why would everything be converted into an integer? As soon as int() is fed with a string containing a '.', it throws a ValueError.


can you keep the name the same as the module name? Keeps confusion down versus `cases.get_case` route exists in the `case` module.


The actual error was here calling origHttpCreateServer instead of origHttpsCreateServer


Case:

threshold: null
realMin: -15000
realMax: 15000,

so combination of negative and positive. It sets threshold to realMin, what causes columns to be drawn from realMin, what does not look correct..


Just noticed that my own PR (#128) has the same problem, investigating.


I'm not sure what you mean. Which event should I use that I don't have to trigger inline?


Well, I think shinano devices use it. At least leo.


Accidentally pushed all changes instead of a small commit (revert in the next commit - please accept this one)


dadasdas


I assume the Case model class exists only for the case_api? If so we can put that class in this file. Otherwise we risk new services adding their models into the "shared" `model.py`.


Right, it won't fail the test but (the atomic counter check will though), I put it as I found it interesting to see the errors in case something fails, what would you recommend instead?


All right. I also another exit do not see


Please Ignore


ok, no need to rename it then


agreed


ok


Oops! I'll fix that tonight.

> On 15 Jun 2015, at 12:54, AliReza Tofighi <notifications@github.com> wrote:
>
> you forget ```php
> global $PL;
>
> —
> Reply to this email directly or view it on GitHub.
>


This Repository is for Data management of Data Specialization Courses at Coursera.


ok, using StringUtils.isBlank(workflowUrl) instead


Has anyone recently changed anything about this code? Because it did not work with the original Is_float() utility function and I had to fix it once Ole reverted his previous commit with df55ee7e5f18d30e2dfba45ee3891d1adb71184e.


Thanks James. Have fun with .NET, we'll see u mid-July. You are the dankest.


which which ?


ok, renamed to fileInputPart


Mmm.. I will keep it as it was for now


Looks like it's a hangover from 08dff4c0b55a2d85f8ac195597c447827f3f4d61, where we flattened the migrations down again.  Edit this commit to remove that INSERT and then drop/create/structure-load your local db to avoid it happening again (just did the same locally and to my PR).


this line is not necessary. the ctor of MForward has encoded the `req` to `msg_bl` already.


Class renamed to HttpResourceDownloader


Whenever you change the DOM. So when you load the modal via ajax, this triggers an event, which can then be picked up by various parts in ui_components.


this line is not necessary.


jabber-net for some reason just cant resolve hostname sometimes, same thing happened to live servers


This change would be useful to me. Does the current state of the master branch get automatically built in to a snapshot somewhere or would I need to build from source to use it? Thanks!


This revision is clean.


i've removed them already :)


And which event would that be?


I agree, I woud rather do it in another commit though


Found this property very useful when iterating through JSON objects, since there is no clean way to do this check otherwise. Figured it doesn't hurt to share, right?

Doesn't check for NaN since NaN isn't supported in JSON anyways.


hola


Doesn't check for NaN since NaN is not supported in JSON anyways.


I'm getting errors on hot reloading, it seems that `JsonpMainTemplate` still uses the callback form of `hotDownloadManifest`.


I agree. I replaced it in all pertinent tests.


What exactly do you mean by testing it on localhost? Are you referring to using mock responses? Or did you do something like setting the Amazon Service URL to localhost? If you need an example of getting order data locally, look at the PHPUnit tests.

Don't worry about the strict standard warning, it has already been fixed in the master branch.


You can pick one, we haven't done this yet ;)


@megastef this should be origHttp**s**CreateServer


Great edits.  Thanks.


Are we are using the term 'vessel' for all parts of the mishkan including the curtains, boards, covers, ropes, etc. ?


Fixed in 061611636beea26b0af6cb27170f859e3da1f207 .


Not sure of this...was in between trying to fix marker's frontend issues.


@cvrebert @mdo: I'd like to merge this  https://github.com/twbs/bootstrap/compare/deps before the stable.

I know the patch version bumps mean nothing since we use tilde, so I can skip the package.json changes as long as everyone remembers to do `rm -rf ./node_modules/ && npm i`. We should really use npm-shrinkwrap for `npm i` too.


@yurem what will happen if acr is not specified in request?


@RohanM can you review this commit before we push to staging? I don't want to undo any of your awesome hard work on performance...


Very minor, but looks like you added an extra space here


GOOD WORK dian...

cm11.0 working camera in autofocus mode view:
https://www.youtube.com/watch?v=2Nefuz1nnvk


Thank you!


im getting whitespace issues when rebasing after this commit


swfObject.js isn't in the develop branch any more


prost selqnin :D


6te te 6ibam pendel


@chelmuth: please add to staging


This is just an upstream merge, it needs to be changed in the gruntfile independently https://github.com/adaptlearning/adapt_framework/blob/develop/Gruntfile.js#L176


Since I won't participate on the next meeting, here are some remarks:

* There is a requirement in the course structure specification that "[a]ll of the languages list MUST be present in a langstring for each title or description element.", so the example should reflect this and only(*) use en, fr, de as lang attributes
* Furthermore I think we should put at least two examples into the course structure: An almost complete example like this one and a most simple example like: https://raw.githubusercontent.com/svrnm/CMI5-CourseStructure-Testing/master/simple/cmi5.xml
* The references objectives should match the existing objectives (uri1 - uri3)
* The uri examples shouldn't be used twice for identifying objectives, AUs and the course

(*) I think we need to change this to "exactly these languages listed MUST be present..."


remove extra blank line here.


do we really need this?


@archetipo per tua info, questa stringa viene stampata così
```
Xml IncorrectModalitaPagamento in line DettaglioPagamentois not defined
```
Se vuoi andare a capo devi mettere ```\n```, altrimenti dovresti aggiungere gli spazi, ad es:
```
                            'Xml Incorrect: '
                            'ModalitaPagamento in line DettaglioPagamento '
                            'is not defined'
```


Shouldn't this be
```
DeadBolt 2 Java - Quick start
```


This should be made to be tolerant of ports not existing in the provided map


SUCCESS and FULL are supposed to be the same?


Hello,
Could you explain to me why you removed some of my last commits (among them:  max_client_buffer, ssl_sites , passive_monitoring) ? Those we not massive changes but very useful to make the puppet module more complete compared to the GUI options.

Regards,
Arnaud


My phone couldn't handle the commit size and chrome crashed
( ° ∆°) }}}

--- Original Message ---

From: "Eric Kolbusz" <notifications@github.com>
Sent: June 15, 2015 3:09 AM
To: "mks66-dw/final" <final@noreply.github.com>
Subject: Re: [final] Merge remote-tracking branch 'origin/master' (3affd81)

i was scrolling so furiously to get to the bottom of this commit that i woke my dog up

i dont even know if any of this is going throguh chrome is lagging too hard

---
Reply to this email directly or view it on GitHub:
https://github.com/mks66-dw/final/commit/3affd81404d802854c630cc35ff7add269ea5d10#commitcomment-11674271


use while ( 1 ) {
see https://devel.rtems.org/wiki/Developer/Coding/Conventions


With this code I also moved the importing of scripts into the try/catch so that if it fails a log will still be produced.


Consider adding some helper functions for this kind of check, like "raspberrypi_mailbox_is_empty()"


Indeed, it is mandatory, added.


Shouldn't you fix the caller that pkg_free's without checking for NULL instead?

If you "allow" it, then you might as well drop all the `if(buf)` before every `pkg_free(buf)`. Allowing both makes the code inconsistent.

Right?


Where is "triangle" ?  "ImportError: No module named triangle "


@amitaibu why after `reject` my code continue executing? In this case i see balalaylka and my app is crashed on the `container.attach` because container is null.


You really only need this if you are not building in default locations, it's pretty confusing (just had someone email me a question) it should be hidden in a subparaph somewhere because it is interesting info but not first paragraph type stuff :)


ل صديقتها والتى تكون اخت البطل اسد . اسد يرى سلوكيات زويا وطبيعتها المتحررة ويغضب منها كثيرا ويظن انها فتاة سيئة النوايا والاخلاق . زويا واسد يتقربان من بعضهما ولكن تتدخل بينهما صديقة اسد والتى تدعى تانفير وتقوم بافساد العلاقة بين اسد وزويا


@andyberry88, would it be better to just log to the user that it took N ms before we were able to write the file?


5 times? not clear why


ditto, "mailbox_is_full()"


Ok, this answers my question in https://github.com/akka/akka/commit/038dedf6679a9c133edc18753e38277eb7327568#commitcomment-11680289
If we expose it as such would be nice to allow users to configure this `5`, as in "max-failure-retries"?


@andreasohlund certainly - if I remember correctly write permissions were to enable all endpoints to be able to send messages to other endpoints by writing to their input queues - it was set up this way to allow everything to "just work" without having to fiddle with security permissions at the endpoint level.


```js
font = new OTFExportController(master, model, masterName).do_export();
```
?


shouldn't need the unsigned int cast here. can add the U to end of the 0xF.


Finally @vilinfield


Thought I tested this but now that I think about it, I don't think it prompted.  Line 66 is wrong for this, so I'll update shortly


Can add U to end of the literal constant to ensure the type is unsigned.


not sure what... but I believe something in this commit broke the service icons on the project details page, they now float in the top left hand corner of the a project details page


make a bit.ly link to your xda thread later
something like bit.ly/falcon
or
bit.ly/ownfalcon


Would that add any value? The reasoning behind the log line was just so a user who wonders why theres a pause for a second or so can see why. But logging at every attempt would be overkill and create too much noise.

Since we're now using the API properly and ensuring the file doesn't exist before this point I expect we can eventually remove this loop and just have `bundleFile.createNewFile()`. I think the only reason we've needed it was to fix a bug that we'd caused by using the API incorrectly.


**Note:** The multiplication has to be on an even numbered register, here we use `2` as we loaded A_ARR into `3`


This (data & 0xFFFFFFF0U) should probably be a helper function also, as it is repeatedly used. I guess it is getting the actual message out of the mailbox?


You are talking about me triggering it myself? The code that loads the modal is in a different project and I would like to avoid having to use anything other than the `ui_component` method on the project's side. So I exposed a function I can still call from ui_components' side, even if inline.

I agree that all of this isn't pretty, but at least I don't overstep any boundaries between the projects. The `renderMarkdown` function is namespaced, so it won't interfere with anything. AFAIK, something like the `load` event isn't triggered for any elements other than `body`, is it?


I have not; I can give that a shot, and see how it affects things.


Oops, I indeed forgot about it, thanks for the reminder. It's not easy to use it here though as this class is uncopyable and I need to return the safe array from a function... so I'm probably not going to change the code to use it.


not needed


Does @reference typeset into something in Doxygen? Or should you just write out a statement about the URL being used as reference documentation?


Hi Elias,

could you please give an example on how to use this in the import/export screen?
I was not able to see an effect.

Thanks,
Udo


If you don't return the IO-instance here, how do you receive the result of the export?
This is why I suggested `return [generator, io];`


@iemejia, if we had sprinkled the code with many `logging.{info,warn,debug}`, we would have catched this kind of problem before it hit the users.


Add space after *


Thanks for the addition. Will this work with the BulkUpdateDescriptor as well?

Eg:-
BulkDescriptor childBulkDescriptor = new BulkDescriptor();

for (int i = 0; i < searchResponse.Hits.Count(); i++)
{
childBulkDescriptor.Update<NestedDocument, object>(u => u
				.Type("document")
				.Id(searchResponse.Hits.ElementAt(i).Id) // document id
				.ScriptFile("my_script")
				.Params(x => x.Add("field_id", fieldIdToUpdate).Add("field_value", fieldValue))
				.RetriesOnConflict(4)
			);
	}


Looks like this commit broke build.
Please see http://ci.cgeo.org/job/c-geo/3823/console to remedy the problem.


should be using xstrdup() here instead


I would like to let the user pass through the command line the numbers he wants to use.


围观


Shell pipelines are not allowed in command calls - the `Popen` call is made with `shell=False`. You'll need to do this in python although to be honest I don't really see the point; what's the harm of running iprconfig if the driver is not loaded?


Make this comment doxygen-parsed, see https://devel.rtems.org/wiki/Developer/Coding/Doxygen


OK, now we have a list for `unit` here.


what is this?


Wicked can run scripts already (since version 0.6.18). But there is a FATE#317974 request regarding OpenVPN.


:ok:


Avoid multiple blank lines in comments, and in code.


@duncanpMS ahh you're too fast, I wanted to commit the same change, and was wondering why I couldn't for a little while :)


This worked fine before your patch. Now it doesn't ;-) (you need to test upstream patches with the current upstream version).


size_t would also be appropriate here.


Ack.


Added tag 1.2.0 for the centos:7 based docker-cleanup. There's a matching tag on Docker hub with image coordinates "meltwater/docker-cleanup:1.2.0"

Also reverted alpine pending investigation. Seems that some docker versions have symbolic links with some shared libraries, strangely enough the alpine verison works just fine with the recent docker 1.6.2 we're using. Perhaps docker 1.6.* is more statically compiled than previous versions.

@tj Would you mind running "ldd /usr/bin/docker" on your host machine and pasting the results in this conversation? What Docker version and host OS are you using?


added favicon he has not appeared


You forgot about removing the gadget directory, please fix it.
I see the problem with returning value of rmdir, but I don't understand how this change fix it. Return value is still USBG_SUCCESS or an error translated from errno. Shouldn't it be solved with adding another USBG_ERROR value for ENOTEMPTY?


Just a minor nitpick (and this applies to other places in the code previously done), I would have written this as `len(units) != 0`, but that's just a matter of taste.


Shouldn'tt his be 0x80000000? Also, add U to end of unsigned literals.


the libc free() (man 3 free) allows free'ing NULL pointers. I think the quality of the codebase is improved if we stick to this standard.

Your suggestion is excellent. We definitely have to remove all `if (buf)` tests!


what ocupped more bytes?
jump line or ; + space?
np b8 xD


take var outside of `if` block


looks good


@aclark4life is this diverging (or intending to) from c.task's definition [1]?

[1] https://github.com/collective/collective.task/blob/master/src/collective/task/profiles/default/workflows/task_workflow/definition.xml


Shoudln't this be 0x80000001? (and U)


:ok: :1234:


Tool: EH-Test

Functionality: Test error handling code of Linux device drivers.

Target drivers: 15 device drivers in Linux-3.17.2.


:D


Well, we would then have to define a clear contract of course:
> Whenever the DOM changes, one has to fire the "domchanged" (or similar) event in order to ensure that     ui_components works properly.

Inline JS is a *really* bad idea, and actually nobody is doing it anymore. See above for the other points. Of course it works, but it does not scale at all if we add more stuff. Also, if we ever decide to use some kind of JS modules and/or async/lazy loading, this will break.


noice!


oh ok, it's not an *exponential* backoff supervisor. Expected `2^n - 1` and was surprised by actual impl for a minute.
Not a real problem though I guess, this is useful as well, :+1:.


I'm confused by this macro. I don't know what kind of structure it intends to be called on, but probably it should be placed closer to the struct definition.


Please remove .DS_Store files


The changes look good.

Do you have a test case (read: an HTML page) that we can incorporate to our test suite where the previous version wasn't grabbing everything that it should, while with your changes we get everything? That would be awesome, since I run the tests automatically before I push them to the repository.

Thanks for your contribution. BTW, if you have any extra contributions (be them refactoring, adding tests or anything else), we'd highly appreciate them. Also, if you have friends that use the project, it would be lovely to get their help, either via testing, submitting bug reports or even patches.

Thank you.


Use Doxygen and fix spacing.


I suppose the thing that worried me about it is that it seems dodgy that we should ever have to wait at all, yet we only log if we had to wait for more than 1 second.


'the them'


Favicons are site-specific and are not deployed with MPOS. No effect on the application whatsoever. Any reason for reviving a 2 year (!) old commit? These comments are spamming inboxes of those people signed up to this repository to track important notifications. Yours is rather ... useless ...


Nevermind - it makes sense as is as well, linearily growing until the max.


@arpanbag1996  make it TheMuppets/proprietary_vendor_google


Perhaps you should use a zero-length array to end this structure, and call it a bcm2835_mailbox_tag? Something like:
```C
struct bcm2835_mailbox_tag {
  uint32_t id;
  size_t buf_size;
  uint32_t buf_len; /* upper bit reserved for request/response indicator */
  uint8_t buf[RTEMS_ZERO_LENGTH_ARRAY];
}
```


add FMRadio here too


remove @@DIFF@@ from Makefile (cmd_munge_script)


See: https://gcc.gnu.org/onlinedocs/gcc/Zero-Length.html


lack of spaces


Again, this macro is hard to understand without all the associated structs being defined.


Delete multiple blank lines.


Does this need to depend on collective.loremipsum's default profile, then? https://github.com/upiq/uu.task/blob/master/uu/task/profiles/default/metadata.xml


> Inline JS is a really bad idea, and actually nobody is doing it anymore.

Says who?

So whenever we load something asynchronously in any project that uses UIComponents, we need to remember to fire a `domchanged` event, because we may or may not rely on it in UIComponents? I don't see how this *scales* any better. By calling the function inline, at least I am in control of what happens from UIComponents' side alone. What is there that would need to *scale*?


an enum might be sensible instead of these macros.


I know, just joking guys


You may want to consider moving this to opts and defaulting to look for `"id"` since once you plan to support nested loading (i.e. comments nested inside a post model) you'll have `something_else_id` and `id` to load within the same request.

Moving it to `opts` lets the developer specify the correct param so you can work on inferring it later if desired.


unmatched `</li>` above must have got here accidentally


Intending to diverge, yes. Assumptions are UU Task needs its own workflow and c.task's workflow was "close enough" to start with.


ok let me do some modifications


@tombentley this is the cause of the new test failing. How did _that_ get in there?


If you use my recommendation above, this goes away. If not, I'd at least do away with this union it adds little of value.


Not likely; c.loremipsum is primarily included here so I can easily add fake users (via @@create-users).


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 549](http://build.prxm.it:8811/viewLog.html?buildId=3024&buildTypeId=Jarvis_DocumentStore_CI) is now running


@ivanlei Overseen it at first, so sorry for coming late with the comment: is this indentation necessary here?


Not completely sure, but I think that '==' is a bashism. Better use '='.


:+1:  It always seems to be the "php7 in the php5.x branch" things that cause problems. The joys of supporting multiple major versions \o/


Move this logic into helper method


INSERT INTO `creature_waypoints` do not use specific dbs


Oops, thanks.


die Klasse hier sollte ein Controler werden


Oh great, thanks for checking!


Move this logic into model and use scope for query


Cool! @Jermolene: Just out of curiosity bit how does the system know that this is a `module-type: macro` and has the tag `$:/tags/Macro` and the title is `$:/edition/tw5.com/dropbox-url.js`? Is there some magic happening with regards to comments as the file header? Thanks.


Follow proper naming conventions.


Added hg:Fort


I'm very sorry, I did not notice that it is an internal method. Yes, it really should behave in the same way as `socket.getfqdn()`.


Sorry, looks like you updated the async iterator after our call. I didn't know it changed.


Oops, please ignore whitespace change at the end


Also, this struct can be equivalently re-defined as
```C
struct bcm2835_mailbox_tag_get_board_version {
  struct bcm2835_mailbox_tag tag_hdr;
  uint32_t rev;
};
```
And you can access the rev field through tag_hdr.buf


(You may need to take care about endianness though.)


It should be in separate method ```date_formatter```


There are already quite a few output=`su ...` constructs accumulated, perhaps consider adding a helper function for that (similar to runasowner).


Везде, где это возможно, вместо место `px` используй `em`, например, унаследованный font-size равен 16px. Тогда для `font-size` вычисляем `18 / 16 = 1.125`; для `border-radius` вычисляем `3 / 18 ~= 0.166667`;. Аналогично поступаем с отступами и полями, в итоге получаем:
```css
padding: 0.222222em 0.444444em 0.222222em 0;
margin-left: -0.5em;
border-radius: 0.166667em;
font-size: 1.125em;
```


@bdmock Hey Bryan, just wanted to ping you that i worked on the locker room and restroom file this weekend.  Make sure to pull down these changes, before you work from your side.  Thanks Much.


:+1: to not pinning versions.


Die Workstation hat später dann auch einen Controler, bin mir nicht sicher, ob wir dazu noch ein extra Model brauchen, eigentlich werden die Daten der Workstation ja kaum übetragen oder geändert, da müsste es reichen, wenn man sie direkt in den Controler mit reinpackt.


Единицы после нуля лучше не писать.
```css
padding: 4px 8px 4px 0;
```


Originally we thought we would have to remove the internal stuff to get the projects signed.  This was a mistake i think he is putting them back.   Would it not have been better to just take a fresh copy of master and apply signing to it.


@array?? Follow proper naming conventions.


주석으로 팔만대장경 집필할 기세


I think a lot of the vc.h code is a bit redundant especially all of the various struct definitions. If you can use the zero-length array representation I suggested, you might be able to unify all of the different approaches by just providing the right size of buf at runtime.

Also, why are the vc macros in the bcm2835 namespace? I'm not opposed to it, but want to be sure we pick an appropriate namespace for macros and functions used in the RPi/RPi2.