Page MenuHomePhabricator
Feed Advanced Search

Apr 22 2017

epriestley added a comment to T12621: remarkup literal block requires text on beginning line.

This is probably the fix:

Apr 22 2017, 4:16 PM · Remarkup, Bug Report
epriestley added a comment to T12621: remarkup literal block requires text on beginning line.

It looks like the existing test cases in master fail on PHP 7.1, presumably because of T10929, so that needs to be fixed before I can fix this.

Apr 22 2017, 4:15 PM · Remarkup, Bug Report
epriestley added a comment to T12621: remarkup literal block requires text on beginning line.

This rule is constructed very awkwardly and I'd sort of prefer to remove it entirely, but I think we can technically fix this without really disrupting things.

Apr 22 2017, 4:09 PM · Remarkup, Bug Report
chad closed T12622: Unhandled Exception ("RuntimeException") when accessing conpherence as Resolved by committing Restricted Diffusion Commit.
Apr 22 2017, 4:01 PM · Bug Report
chad added a project to T12621: remarkup literal block requires text on beginning line: Remarkup.
Apr 22 2017, 2:34 PM · Remarkup, Bug Report
chad added a revision to T12622: Unhandled Exception ("RuntimeException") when accessing conpherence: Restricted Differential Revision.
Apr 22 2017, 2:30 PM · Bug Report
nevogd updated the task description for T12622: Unhandled Exception ("RuntimeException") when accessing conpherence.
Apr 22 2017, 2:10 PM · Bug Report
nevogd created T12622: Unhandled Exception ("RuntimeException") when accessing conpherence.
Apr 22 2017, 1:47 PM · Bug Report
lambdafu updated the task description for T12621: remarkup literal block requires text on beginning line.
Apr 22 2017, 1:35 PM · Remarkup, Bug Report
lambdafu created T12621: remarkup literal block requires text on beginning line.
Apr 22 2017, 1:32 PM · Remarkup, Bug Report
chad added a comment to T12613: In diffusion hosting subversion repository, several commands do not return (freeze)..

fails for my MacBook:

Apr 22 2017, 5:57 AM · Diffusion, Bug Report
k_kazu added a comment to T12613: In diffusion hosting subversion repository, several commands do not return (freeze)..

Thank you for replay.
Umm..., Reproduce with the same procedure on my environment.
It may depend on the client environment.

Apr 22 2017, 2:11 AM · Diffusion, Bug Report

Apr 21 2017

epriestley added a comment to T12619: Conpherence fatals on a fresh install.

No problem, thanks for the report!

Apr 21 2017, 10:06 PM · Conpherence, Bug Report
bbrdaric added a comment to T12619: Conpherence fatals on a fresh install.

Thanks for a fast response!

Apr 21 2017, 10:04 PM · Conpherence, Bug Report
epriestley closed T12619: Conpherence fatals on a fresh install as Resolved by committing Restricted Diffusion Commit.
Apr 21 2017, 9:50 PM · Conpherence, Bug Report
chad added a revision to T12619: Conpherence fatals on a fresh install: Restricted Differential Revision.
Apr 21 2017, 9:48 PM · Conpherence, Bug Report
epriestley added a revision to T12619: Conpherence fatals on a fresh install: Restricted Differential Revision.
Apr 21 2017, 9:47 PM · Conpherence, Bug Report
epriestley claimed T12619: Conpherence fatals on a fresh install.
Apr 21 2017, 9:47 PM · Conpherence, Bug Report
bbrdaric created T12619: Conpherence fatals on a fresh install.
Apr 21 2017, 9:39 PM · Conpherence, Bug Report
chad closed T12604: Pager hard to notice in Diffusion as Resolved by committing Restricted Diffusion Commit.
Apr 21 2017, 6:43 PM · Design, Bug Report, Diffusion
chad added a revision to T12604: Pager hard to notice in Diffusion: Restricted Differential Revision.
Apr 21 2017, 6:32 PM · Design, Bug Report, Diffusion
chad added a revision to T12604: Pager hard to notice in Diffusion: Restricted Differential Revision.
Apr 21 2017, 6:17 PM · Design, Bug Report, Diffusion
chad renamed T12614: Add file transform unit tests from Error after upgrade Phabricator to Add file transform unit tests.
Apr 21 2017, 5:43 PM · Bug Report
epriestley added a comment to T12614: Add file transform unit tests.

Adding the file should fix this error, but we'll almost certainly break it again when we go to 800x800 without a test.

Apr 21 2017, 3:07 PM · Bug Report
epriestley added a comment to T12614: Add file transform unit tests.

Basically, we routinely double the size of transforms. When we do, we also need to replace default-111x111.png with default-222x222.png so we have a fallback. We have failed to do this ~every time we've doubled things since it's impossible to know/remember, so getting test coverage seems like a good idea.

Apr 21 2017, 3:07 PM · Bug Report
chad added a comment to T12614: Add file transform unit tests.

oh i see... so I just make one for now?

Apr 21 2017, 3:06 PM · Bug Report
epriestley added a comment to T12614: Add file transform unit tests.

In D17295, we changed the "profile" transform to generate 400x400px images instead of 200x200px images. When we add or change a transform, we need to include a default result file for that transform which can be used as a fallback if the transform fails (most often because an install does not have gd, but possibly because of a corrupt image, inaccessible file storage, etc). That change didn't update the default result file, resources/builtin/image-200x200.png, so we were left with a 400x400px transform but a 200x200px unused default image that were not linked to one another.

Apr 21 2017, 3:05 PM · Bug Report
chad added a comment to T12614: Add file transform unit tests.

Er, sorry, what mistake did we make again? I can’t find 400x400 ever
existing?

Apr 21 2017, 2:59 PM · Bug Report
epriestley added a comment to T12614: Add file transform unit tests.

I think the upstream actions are probably:

Apr 21 2017, 2:32 PM · Bug Report
chad added a comment to T12614: Add file transform unit tests.

I don't see a 400x400.png anywhere in builtin history.

Apr 21 2017, 2:30 PM · Bug Report
epriestley added a comment to T12614: Add file transform unit tests.

I suspect the reproduction steps are something like:

Apr 21 2017, 2:29 PM · Bug Report
chad added a comment to T12614: Add file transform unit tests.

How can we reproduce this locally?

Apr 21 2017, 2:23 PM · Bug Report
epriestley moved T11648: Jumping to an reply inline comment via URL anchor should highlight the comment node accurately from Backlog to Glitches on the Inline Comments board.
Apr 21 2017, 12:23 PM · Inline Comments, Differential, Bug Report
epriestley moved T11784: Differential "stick to the comment area" and "jump to anchor" scroll behaviors interact destructively while loading content from Backlog to Glitches on the Inline Comments board.
Apr 21 2017, 12:22 PM · Inline Comments, Differential, Bug Report
epriestley moved T12153: 'Show Hidden Comments' bubble does not fit in margin next to line numbers >1000 from Backlog to Glitches on the Inline Comments board.
Apr 21 2017, 12:22 PM · Inline Comments, Differential, Bug Report
epriestley added a project to T11648: Jumping to an reply inline comment via URL anchor should highlight the comment node accurately: Inline Comments.
Apr 21 2017, 12:20 PM · Inline Comments, Differential, Bug Report
epriestley added a project to T12153: 'Show Hidden Comments' bubble does not fit in margin next to line numbers >1000: Inline Comments.
Apr 21 2017, 12:20 PM · Inline Comments, Differential, Bug Report
epriestley added a project to T11784: Differential "stick to the comment area" and "jump to anchor" scroll behaviors interact destructively while loading content: Inline Comments.
Apr 21 2017, 12:20 PM · Inline Comments, Differential, Bug Report
epriestley moved T12109: Updating revision reverts description changes, if you edit the description while `arc` is running from Branch Handling to Far Future Work on the Differential board.
Apr 21 2017, 11:23 AM · Differential, Bug Report
epriestley added a comment to T12613: In diffusion hosting subversion repository, several commands do not return (freeze)..

I can't reproduce this:

Apr 21 2017, 11:09 AM · Diffusion, Bug Report
kevinchea created T12614: Add file transform unit tests.
Apr 21 2017, 7:05 AM · Bug Report
chad placed T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned up for grabs.
Apr 21 2017, 4:33 AM · Dashboards, Bug Report
k_kazu updated the task description for T12613: In diffusion hosting subversion repository, several commands do not return (freeze)..
Apr 21 2017, 3:45 AM · Diffusion, Bug Report
k_kazu created T12613: In diffusion hosting subversion repository, several commands do not return (freeze)..
Apr 21 2017, 3:32 AM · Diffusion, Bug Report

Apr 20 2017

chad renamed T12604: Pager hard to notice in Diffusion from Can't find folder in Diffusion to Pager hard to notice in Diffusion.
Apr 20 2017, 8:04 PM · Design, Bug Report, Diffusion
jmeador added a comment to T12604: Pager hard to notice in Diffusion.

I agree, pagination way below the fold is rather inconspicuous

Apr 20 2017, 5:59 PM · Design, Bug Report, Diffusion
chad added a comment to T12604: Pager hard to notice in Diffusion.

hahaha, thanks... really easy to miss those

Apr 20 2017, 5:58 PM · Design, Bug Report, Diffusion
chad added a project to T12604: Pager hard to notice in Diffusion: Design.
Apr 20 2017, 5:58 PM · Design, Bug Report, Diffusion
jmeador added a comment to T12604: Pager hard to notice in Diffusion.

Bystander input: try page 3: https://phabricator.kde.org/source/krita/browse/master/libs/ui/?offset=200

Apr 20 2017, 5:57 PM · Design, Bug Report, Diffusion
chad created T12604: Pager hard to notice in Diffusion.
Apr 20 2017, 5:38 PM · Design, Bug Report, Diffusion
epriestley closed T12603: Exception visible when sshing to phacility as Resolved.

I patched this and deployed it. I'll follow up on the internal report with details.

Apr 20 2017, 1:37 PM · Phacility, Bug Report
epriestley added a comment to T12603: Exception visible when sshing to phacility.

Thanks, this is fallout from an incident last night. The particulars involved instance and user details so the discussion was kept private. I'll fix this shortly.

Apr 20 2017, 1:33 PM · Phacility, Bug Report
avivey created T12603: Exception visible when sshing to phacility.
Apr 20 2017, 7:47 AM · Phacility, Bug Report
• sshannin created T12602: Assignees of tasks don't count as subscribers in maniphest search.
Apr 20 2017, 1:12 AM · Maniphest, Bug Report

Apr 19 2017

chad added a comment to T12594: Scrolling to the right-hand side locks the mouse arrow to the orizonatal scroll bar and prevents navigation.

(there are some other wierd issues in IE11, so I expect I'll clean up some of the UI issues I see).

Apr 19 2017, 5:01 PM · Bug Report
chad added a comment to T12594: Scrolling to the right-hand side locks the mouse arrow to the orizonatal scroll bar and prevents navigation.

We require reporters file complete steps to reproduce the issue. If we can't based on the information provided, it's still on you to provide an updated, more specific set of steps. Also, we don't support third party extenstions like sprint. Feel free to reproduce against a clean install on phacility.com

Apr 19 2017, 5:00 PM · Bug Report
RikiWiki added a comment to T12594: Scrolling to the right-hand side locks the mouse arrow to the orizonatal scroll bar and prevents navigation.

is there something I can do to help you? or some suggestions regarding the browser that better supports the tool?

Apr 19 2017, 4:48 PM · Bug Report
chad added a comment to T12594: Scrolling to the right-hand side locks the mouse arrow to the orizonatal scroll bar and prevents navigation.

I can't reproduce this.

Apr 19 2017, 4:45 PM · Bug Report
RikiWiki added a comment to T12594: Scrolling to the right-hand side locks the mouse arrow to the orizonatal scroll bar and prevents navigation.

On the 26th we will update the version to the new one, I have been told.

Apr 19 2017, 3:19 PM · Bug Report
RikiWiki added a comment to T12594: Scrolling to the right-hand side locks the mouse arrow to the orizonatal scroll bar and prevents navigation.

TEST:
Version Information

Apr 19 2017, 3:05 PM · Bug Report
epriestley added a comment to T12592: User awaiting approval has no image.

For my own reference, T12570 isn't exactly related but can probably be tested/fixed at the same time.

Apr 19 2017, 2:35 PM · Bug Report, Auth
epriestley claimed T12592: User awaiting approval has no image.

Stealing this since I think it's session/auth related, not profile picture related: I think the user has a legitimate picture, they just aren't allowed to download it since they have a partial session.

Apr 19 2017, 2:35 PM · Bug Report, Auth
epriestley added a comment to T12592: User awaiting approval has no image.

T8918 is related.

Apr 19 2017, 2:33 PM · Bug Report, Auth
chad added a comment to T12594: Scrolling to the right-hand side locks the mouse arrow to the orizonatal scroll bar and prevents navigation.

Please read and follow Contributing Bug Reports. Specifically this does not state which version of Phabricator you are running.

Apr 19 2017, 2:18 PM · Bug Report
RikiWiki created T12594: Scrolling to the right-hand side locks the mouse arrow to the orizonatal scroll bar and prevents navigation.
Apr 19 2017, 1:51 PM · Bug Report
arcadien added a comment to T5229: Fixing up invalid encoding confuses archanists utf8-check.

Is Arcanist saving information about files somewhere? or is it just a problem with the utf8 validator?

Apr 19 2017, 1:10 PM · Bug Report, Arcanist
chad created T12592: User awaiting approval has no image.
Apr 19 2017, 4:29 AM · Bug Report, Auth

Apr 18 2017

chad closed T12581: The "Recent Activity" feed flooded with duplicates because of invalid UTF-8 characters as Invalid.

Please read and follow Contributing Bug Reports and Providing Reproduction Steps.

Apr 18 2017, 8:48 PM · Bug Report
amckinley closed T10828: File deletion should be queued up and run by daemons as Resolved by committing Restricted Diffusion Commit.
Apr 18 2017, 6:09 PM · Contributor Onboarding, Daemons, Files, Bug Report
epriestley closed T12585: Unable to create differential as Invalid.

Bug reports require reproduction steps. See Contributing Bug Reports and Providing Reproduction Steps.

Apr 18 2017, 1:44 PM · Bug Report
atul.agrawal9911 added a comment to T12585: Unable to create differential .

This is happening because my project directory not contains .jshintrc file

Apr 18 2017, 12:46 PM · Bug Report
atul.agrawal9911 added a comment to T12585: Unable to create differential .

my .arclint file

Apr 18 2017, 12:23 PM · Bug Report
atul.agrawal9911 created T12585: Unable to create differential .
Apr 18 2017, 12:22 PM · Bug Report
• 94else merged task T12583: test into T12582: test.
Apr 18 2017, 7:21 AM · Bug Report
• 94else created T12583: test.
Apr 18 2017, 7:19 AM · Bug Report
• 94else added a comment to T12581: The "Recent Activity" feed flooded with duplicates because of invalid UTF-8 characters.

home_aboutus_nor@2x.png (104×104 px, 491 KB)

Apr 18 2017, 7:15 AM · Bug Report
tavvva added a comment to T12581: The "Recent Activity" feed flooded with duplicates because of invalid UTF-8 characters.

The attached patch seems to fix the issue ... hopefully with no drawbacks. We gonna test that for few weeks.

Apr 18 2017, 7:02 AM · Bug Report
tavvva created T12581: The "Recent Activity" feed flooded with duplicates because of invalid UTF-8 characters.
Apr 18 2017, 6:47 AM · Bug Report
chad closed T12579: ./bin/mail show-outbound command should type check <id> as Invalid.
Apr 18 2017, 5:46 AM · Mail, Contributor Onboarding
atul.agrawal9911 added a comment to T12579: ./bin/mail show-outbound command should type check <id>.

It should give me warning if i am using string value for my id part.

Apr 18 2017, 5:43 AM · Mail, Contributor Onboarding
chad added a comment to T12579: ./bin/mail show-outbound command should type check <id>.
chad@phac-dev:/var/www/html/dev/phabricator$ ./bin/mail show-outbound --id 1
PROPERTIES
ID: 1
Status: sent
Related PHID: PHID-USER-hh5jvrt4dfkdzciswub6
Message:
Apr 18 2017, 5:42 AM · Mail, Contributor Onboarding
atul.agrawal9911 added a comment to T12579: ./bin/mail show-outbound command should type check <id>.

i am trying this command
./bin/mail show-outbound --id <my-email-id>

Apr 18 2017, 5:42 AM · Mail, Contributor Onboarding
chad added a comment to T12579: ./bin/mail show-outbound command should type check <id>.

Please provide complete steps to reproduce the error. Based on the information above I cannot reproduce anything.

Apr 18 2017, 5:40 AM · Mail, Contributor Onboarding
atul.agrawal9911 created T12579: ./bin/mail show-outbound command should type check <id>.
Apr 18 2017, 5:38 AM · Mail, Contributor Onboarding

Apr 17 2017

joshuaspence updated the task description for T12578: Herald rule for "Differential reviewers do not include" didn't fire when I expected it to.
Apr 17 2017, 11:15 PM · Differential, Herald, Bug Report
joshuaspence added a comment to T12578: Herald rule for "Differential reviewers do not include" didn't fire when I expected it to.

Hmm ok. I think that I'm not usually listed explicitly as a reviewer, which means that Reviewers: #some_project_that_i_am_a_member_of doesn't include me. Would you be in favor of adding a new [Differential was reviewed by] field to Herald?

Apr 17 2017, 10:56 PM · Differential, Herald, Bug Report
epriestley added a comment to T12578: Herald rule for "Differential reviewers do not include" didn't fire when I expected it to.

I don't think this behavior has changed. This has always meant "reviewers", not "reviewers who accepted the revision".

Apr 17 2017, 10:50 PM · Differential, Herald, Bug Report
joshuaspence created T12578: Herald rule for "Differential reviewers do not include" didn't fire when I expected it to.
Apr 17 2017, 10:48 PM · Differential, Herald, Bug Report
amckinley claimed T10828: File deletion should be queued up and run by daemons.
Apr 17 2017, 7:26 PM · Contributor Onboarding, Daemons, Files, Bug Report
NDKilla added a comment to T12575: All-day reccurring events reoccurr for two days instead of one.

@chad Like I said I'm aware you guys don't support prototype apps, figured I'd just make someone aware of it, not looking to make it work right now.

Apr 17 2017, 5:59 PM · Calendar, Bug Report
chad added a comment to T12575: All-day reccurring events reoccurr for two days instead of one.

Oh, Calendar is a prototype. :/

Apr 17 2017, 5:04 PM · Calendar, Bug Report
chad added a comment to T12575: All-day reccurring events reoccurr for two days instead of one.

Do you mind verifying the bug on an up-to-date version of Phabricator?

Apr 17 2017, 5:03 PM · Calendar, Bug Report
NDKilla created T12575: All-day reccurring events reoccurr for two days instead of one.
Apr 17 2017, 5:00 PM · Calendar, Bug Report
epriestley added a comment to T12232: Phortune "add payment method" erroring out on submission.

You are now officially Pay2Win.

Apr 17 2017, 2:54 PM · Phortune, Bug Report
yelirekim added a comment to T12232: Phortune "add payment method" erroring out on submission.

Yeah I just wanted to buy myself a badge.

Apr 17 2017, 2:45 PM · Phortune, Bug Report
epriestley closed T12232: Phortune "add payment method" erroring out on submission as Resolved.

Alright, I'm going to assume this was cache/ghosts until it happens to someone else since it seems like no reproduction case exists anywhere anymore.

Apr 17 2017, 2:44 PM · Phortune, Bug Report
yelirekim added a comment to T12232: Phortune "add payment method" erroring out on submission.

No nothing like that but FWIW, it just worked when I tried to reproduce it again.

Apr 17 2017, 2:41 PM · Phortune, Bug Report
epriestley added a comment to T12232: Phortune "add payment method" erroring out on submission.

Maybe a simpler empirical test is, on the "Add Payment Method" page, does this work in the developer console?

Apr 17 2017, 2:41 PM · Phortune, Bug Report
epriestley added a comment to T12232: Phortune "add payment method" erroring out on submission.

(Also, do you run some browser extension like "disable all external Javascript so ads don't work"?)

Apr 17 2017, 2:40 PM · Phortune, Bug Report
yelirekim added a comment to T12232: Phortune "add payment method" erroring out on submission.

Yeah.

Apr 17 2017, 2:38 PM · Phortune, Bug Report