text:
'Permission does not exist', 400

Laravel Version:  5.6.18
Voyager Version: 1.1.0
PHP Version:7.1.10
Database Driver & Version:MySQL 5.7.19

Description:
I can only brows the default BREAD but cant not edit
I can not Add or Edit the BREAD  for any taable
/** @deprecated */
    public function can($permission)
    {
        $this->loadPermissions();

        // Check if permission exist
        $exist = $this->permissions->where('key', $permission)->first();

        // Permission not found
        if (!$exist) {
            throw new \Exception('Permission does not exist', 400);
        }

        $user = $this->getUser();
        if ($user == null || !$user->hasPermission($permission)) {
            return false;
        }

        return true;
    }

    /** @deprecated */
    public function canOrFail($permission)

"Permission does not exist"
Steps To Reproduce:

Its here
Seeded from here
Called from install-command here
Granted to admins created by voyager:admin here

Ok got it. I was run composer update previously, and it updates Voyager from version v1.0.11 to v1.1.1 without changing the PermissionsTableSeeder.php. Fresh install correctly replacing the file and the permission is there.
Thanks.

Yep, it's mentioned in the upgrade guide that you need to manually add browse_bread (or do it your way)

I got the same, please help. It deleted all of my backoffice

I sincerely doubt "it" didn't "delete your backoffice".
In either case, this issue has been closed for over a month.  Please open your own or ask on our slack

authors: PointfullDevelopers emptynick fletch3555 kayrules rddewan


text:
Feature/refactor

@the-rango please review and approve.

Hi @emmohac!
Thanks so much for taking the time to go through and clean up some of this code.
Unfortunately, last summer we started a massive rewrite that restructured the entire project. main has diverged significantly since then. As a result we will be unable to merge these changes.

source: gharchive/pull-request

repo: the-rango/AntAlmanac

license: MIT

license_type: permissive

license_source: github-api


text:
Parameter for Snapshot-Lab

Would it be possible to have a parameter added to allow custom naming of the snapshots for the Snapshot-Lab command? I'd like to create a temporary snapshot after each module.

You can create snapshots at any time with the Hyper-V cmdlets. I'd say the Snapshot-Lab command is a convenience and makes it easier to roll back the entire lab. I think what you really want is a Snapshot-Lab [-snapshotname ] and a corresponding Rollback-Lab [-snapshotname ]. I say go for it and submit a PR!

If you can point me to where the code for those commands is located, I'll work on it after my course is done. I have a workaround now using the Hyper-V cmdlets.

https://github.com/theJasonHelmick/PS-AutoLab-Env/blob/master/Tools/PSAutoLab/PSAutoLab.psm1

Those changes look pretty easy to make since the underlying commands already take a parameter value for a snapshot name. All we (you) need to do is add a SnapshotName parameter and set a default value using the current setting. That won't break anything and still allow us to snapshot at anytime with a new name. Although, now that I think about it, there would need to be some logic to check if there is an existing snapshot with the same name.

Got it!
I'll see what I can come up with.
Thanks!
Michael Bender
-via mobile device

From: Jeff Hicks notifications@github.com
Sent: Thursday, December 15, 2016 1:00:29 PM
To: theJasonHelmick/PS-AutoLab-Env
Cc: Michael Bender; Author
Subject: Re: [theJasonHelmick/PS-AutoLab-Env] Parameter for Snapshot-Lab (#72)
Those changes look pretty easy to make since the underlying commands already take a parameter value for a snapshot name. All we (you) need to do is add a SnapshotName parameter and set a default value using the current setting. That won't break anything and still allow us to snapshot at anytime with a new name. Although, now that I think about it, there would need to be some logic to check if there is an existing snapshot with the same name.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/theJasonHelmick/PS-AutoLab-Env/issues/72#issuecomment-267411232, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALD9TJCzQ9noueQdcC9CzXOauYBpydv_ks5rIY5NgaJpZM4LNHv2.

Status? Currently working on new version

Dropped the ball and forgot about this. I will take a look and see what I can come up with. Won't be till last week of June as I'm offline most of the next two weeks.

Actually my friend, don't worry about it. I got it on my list so I will explore it for the future release.

source: gharchive/issue

repo: theJasonHelmick/PS-AutoLab-Env


text:
add http://101.ru/radio/channel/43 station

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

station added


text:
allow usage of OpenCombine via OpenCombineShims

this makes CombineSchedulers work with OpenCombine, and more specifically, OpenCombineShims, allowing to use Combine on supported platforms and OpenCombine anywhere else.
This helps in the transition of TCA to windows, allowing to build with OpenCombine for now.
While we have the external PR open, we can update our internal develop branch to have the windows code and start collapsing our internal stack since the Pointfree folks might take a bit to evaluate our approach.

External PR https://github.com/pointfreeco/combine-schedulers/pull/90

Should fix WPP-171

repo: thebrowsercompany/combine-schedulers


text:
Feature Request: SEED: Bird Auto Hunting

Please add auto hunting feature

hello  , we have no planned for this feature .

repo: thebrumby/HotWalletClaimer


text:
Standalone edition for Mac always returns as HTML as format response type.

Only on the standalone edition of Postman the responses always format to HTML instead of the response type. (JSON, XML). This is extremely annoying when working with many APIs.
The issues doesn't exist in the chrome web store version.

How is this an issue for apiary2postman?

Whoops

authors: thecopy voidstarfire

license: mit

license_source: bigquery


text:
Support intermediate nested levels

This should fix #261. As noted in the commit comments, there may be room for improvement. I don't think this should be merged just yet, personally, and I would appreciate a review from anyone else who might have some insights.
I implemented for flexibility between flatten-with-intermediate-levels-preserved and flatten-without-intermediate-levels-preserved, but I'm not 100% sure if that's necessary. I'm not happy that the validate --> validateModel path flattens twice, but I was unable to get the tests to pass without that. If anyone has any suggestions for improving the flow, please leave a comment.
Please also let me know if the unit tests need improvement.

Oh, and I should add I wasn't sure if I was expected to commit the results of grunt rig or grunt uglify or if that would be done separately (maybe on release). So far, I have not committed those, only the raw source and test changes. Please let me know if I should add commits for the other stuff. (Also, it might be worth updating README.md to include info on this preferred practice.) Thanks!

Hi, i  think that is not necessary the preserve argument, so  i suggest  always flatten nested levels
I also think that complex objects should be added to result  as is today   since they can be validated

As for the files, you put in the PR the right ones

Okay. I'll add another commit to remove that flag. I also have a few minor tweaks I want to make to the unit tests. I'll have a commit added today at some kind.

Waiting your commit, @platinumazure.

Apologies, ended up unexpectedly busy. Hoping to do it in the next few hours, or else for sure tomorrow.
Anyone have any idea how to avoid the double flatten in the validate --> validateModel flow? I could still use some thoughts there.

I would fix the nested issue here and open another issue to discuss the flatten duplication.

Alright, good call. I'll do that.
I figured out what was wrong, incidentally. The problem is that we shouldn't be flattening at all; rather, we should interpret dotted-properties in the validation object as (potential) multi-level accesses and use a combination of Array#split and _.reduce to actually index the properties on the model. This would only need to be done once per dotted-property and we wouldn't need to waste time constructing a flattened object. But as suggested, I will put that in a separate pull request.
I'll work on that commit now and have it in soon.

FYI: This may also fix #239.

In the mean time i managed to remove the duplicate call to flatten. See #266

Hey @platinumazure, could you update the README and versioning specifications to 0.11.1? Tomorrow we'll be launching the 0.11.0 version and just after it, we'll be merging yours.

I can, but wouldn't it be better to let the collaborators do it in their own pull request? If there are unforeseen issues with merging or delays on my end, then I'd need to change the version again. It seems to me that rank-and-file pull request openers like me shouldn't worry about what version this goes in.
Since you're releasing 0.11.0, do you need me to rebase this request again?

I think this is ready to go. Probably it will not be necessary to rebase.
I just would cleanup the comments and remove the extra variable reminiscent from preserveLevels option implementation (see comments in code)

Fixed the comment. The pull request says auto-merge is still possible, so I won't rebase unless requested.
I've skipped the version change because, as I explained earlier, it seems to me that the version should only be changed by the collaborators when a release time is chosen. Yes, I know you've told me you were planning to release, but if you changed your mind, my adding a version change would make this pull request much harder to merge. Apologies for the inconvenience but I'm just trying to follow best practices.

@platinumazure No problem, man. I'll be merging this today to version 0.11.1. Thank you so far! :+1:

Merged! Thanks! :+1:

Sorry for the mess in the commit history — I've made a mistake as pointed out in #270.
Thank you for your attention so far. v0.11.1 is released! :+1: :beers:

One last thing: @platinumazure if you find an issue reporting something that this PR resolves, please, ping me out loud to further closing and referencing.

authors: blikblum chiefGui platinumazure


text:
rename task.spawn -> task.run()

Motivation
There is a more complete writeup and discussion for this change in #360 but the long and the short are that task.spawn() the function and standalone spawn() the operation have different semantics and should be named different things.
Approach
This renames task.spawn() to task.run() which is in keeping with the static definition of run(). Global run() is a function that turns an operation into a task as the child of the root task, there fore task.run() is a function that turns an operation into a task as the child of task.
task.spawn() is deprecated in favor of task.run().

You could add the jsdoc @deprecated comment on spawn.
  /** @deprecated Use run instead */
  spawn<R>(operation?: Operation<R>, options?: TaskOptions): Task<R>;

Editors like vscode will put a line through on anything marked with it

repo: thefrontside/effection


text:
Consider prefetching article images before marking them as loaded

Consider that for an article or text page, before we declare it in the ready state (and finish all related animations / transitions), we might want to prefetch the images. The advantage of this is the experience will be smoother and there will be no bouncing images pushing the redraws on the page. The disadvantage of this is the text is delayed until all the images are ready. What are our thoughts?

I guess especially know we have our loading animation, it would work really well, right? Even though text comes slower.

Yeah. I mean I think the whole images loading and pushing the text of your
page down is a shitty experience and part of why apps feel better than
websites.
On Sun, Aug 7, 2016, 1:16 PM Emil Goldsmith Olesen notifications@github.com
wrote:

I guess especially know we have our loading animation, it would work
really well, right? Even though text comes slower.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thegazelle-ad/gazelle-front-end/issues/38#issuecomment-238076735,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADAFzYPSKOhsnwunt1_-Gf91vW4J61OWks5qdb6WgaJpZM4Jbvuo
.

Yeah, let's do that. I definitely agree it's a cool experience having everything show up. And with our animation it doesn't seem like it's reacting slowly, just that it's working a little bit. Or nearly just like it's instant, just with a cool transition

Instead we could use "Progressive" JPGs instead

Closing this for now, if we find the need for it later we can open a new issue

authors: Goldsmith94 emilgoldsmith lingz zanemountcastle

repo: thegazelle-ad/gazelle-server


text:
ipv6 support

Add __meta_pve_ipv4 and __meta_pve_ipv6 labels
Both can be false if no matching IP was detected.
For ipv6 not only loopback IP addresses are skipped, but also local link adresses, since prometheus might not be running on the same network as the vms.
Replaces #149

BREAKING CHANGE: The label __meta_pve_ip was removed in favor of __meta_pve_ipv4 and __meta_pve_ipv6.

For now I've left __meta_pve_ip in there, for backwards compatibility, but it can also be removed, if a breaking change is feasible.

Would do a major release bump, personally I would prefer to keep it clean.

Thanks again for your work.

You're welcome
I'm looking forward to the next release, so I can get rid of some of my workarounds :)

Tagged now, should be available in a couple of minutes :)

authors: peschmae xoxys

repo: thegeeklab/prometheus-pve-sd


text:
🛑 dynamiteevents.co.uk/ is down

In 73ca9d2, dynamiteevents.co.uk/ (https://dynamiteevents.co.uk/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: dynamiteevents.co.uk/ is back up in 3d260e5.

repo: thegreatgulper/checksitestatus


text:
🛑 dynamiteevents.co.uk/ is down

In b655b4f, dynamiteevents.co.uk/ (https://dynamiteevents.co.uk/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: dynamiteevents.co.uk/ is back up in 014c4c5.


text:
[FEATURE] Users can add new submissions

User story:
Given the feature toggle is turned on,
as an authenticated user
when I create a new submission,
then a multi-step form guides me through the process, validating my inputs and saving my submission upon completion.
Acceptance criteria:

authenticated user with the feature flag turned on can access /submissions/new (name to be decided)
multi-part form is displayed,
upon completion of each step, the data is being validated,
upon completion of the last step, the data is persisted in the database,
status of the submission changes accordingly (in progress -> in review, or similar)

Developer notes:

The focus of this task is to make the multi-step form functional, not pretty. Details on the front-end and styling will be handled independently at a different time. Instead, this task aims to do all the heavy lifting with regard to data validation.
We can utilize django-formtools to implement the form wizard (i.e. to split a form across multiple web pages).
It's possible to store the data from a started, but not completed submission in a server-side session. This would allow the user to abandon their submission and then come back to it, without loosing the data (as long as their session cookie is still valid)
I expect that by working on this task we will discover many potential edge cases, specifically about what to display under certain conditions. As a part of this task I suggest documenting all findings to have a starting point for discussions, which in turn should enable us to define more fine-grained tasks to fix specific cases.

For future reference, this piece of information was crucial to make the file upload functional: https://docs.djangoproject.com/en/4.1/ref/forms/api/#binding-uploaded-files

repo: thegreenwebfoundation/admin-portal


text:
Requesting script for LIGER dataintegration on ATAC-seq data

Hallo, I wanna ask if its possible for you to upload the code used for data-integration of ATAC-sequencing data with LIGER :)

Hi @AndreasHHaugaard,
Please check out the scib-pipeline repo for the code for all integration runs. As all integration runs were done with the same pipeline (for scRNA-seq and scATAC), we don't have a separate script for a particular integration run.

authors: AndreasHHaugaard LuckyMD

repo: theislab/scib-reproducibility


text:
Fix:  buttons stuck in dark theme

Fixes regression from a5cd87a2bfed6ce25c39969be40
current:

expected:

In lieu of just removing the class and or reverting completely, i tried to figure out what happened so i could restore that code… but,
since it wasn't an outright regression nor a granular documented change, it was pretty unclear to me what the original intent was so I checked what Flowbite uses[^0] and reverted the base lightness accordingly to 300
it did occur to me that it was possibly an intended change to have the buttons dark always… there are some styles like that alongside "light" mode tables. however if this was the case, it should be implemented differently as the dark class cannot be overridden at the pagination container level which makes workaround and customization tedious- which is surely not the original intent, of course.
[^0]: the color shades (300/700) are the same although Flowbite relies on border-color instead of divide-[color]

This is according to the original flowbite.com design.
Please see https://flowbite.com/docs/components/pagination/.

This is not the same as original flowbite

This is according to the original flowbite.com design. Please see https://flowbite.com/docs/components/pagination/.

it did occur to me that it was possibly an intended change to have the buttons dark always… there are some styles like that alongside "light" mode tables. however if this was the case, it should be implemented differently as the dark class cannot be overridden at the pagination container level which makes workaround and customization tedious- which is surely not the original intent, of course.

authors: binury shinokada

repo: themesberg/flowbite-svelte


text:
Error while using function

This is my code that gets list of products from database
LLPhant\Chat\FunctionInfo\TypeMapper::mapPhpTypeToJsonSchemaType(): Argument #1 ($reflectionType) must be of type ReflectionNamedType, null given
My code is
$functionInfo = FunctionBuilder::buildFunctionInfo(new Products(), 'indexProducts');
                $this->chat->addFunction($functionInfo);
                $response = $this->chat->generateTextOrReturnFunctionCalled($prompt);

Hey @absolutemadla8,
Can you give me the signature of your method ?

Maybe you didn't put a type on one of your function parameter.
Working example: public function sendMail(string $subject, string $body, string $email)

You can describe manually your method if you prefer:
$subject = new Parameter('subject', 'string', 'the subject of the mail');
$body = new Parameter('body', 'string', 'the body of the mail');
$email = new Parameter('email', 'string', 'the email address');

$function = new FunctionInfo(
    'sendMail',
    new MailerExample(),
    'send a mail',
    [$subject, $body, $email]
);

What I'm trying to do is when a user asks about available products

did you try what I suggested and still got the error ?

did you try what I suggested and still got the error ?

Hey, its working now. I forgot to put type on one of my parameters.
However I'm confused how to differentiate between arguments that I want to pass internally and arguments that I want ask the user. Such as products connected to a store_id has to be passed in internally by my logic.

Actually better question would be how can I create my own tool using this library?
This would be really helpful. Thanks!

Just one last thing, while the function is being successfully executed I am unable to pass the response to GPT. Therefore the function is executed but there is not message sent to the user with the data that was given as a response by the function.

Hey,

However I'm confused how to differentiate between arguments that I want to pass internally and arguments that I want ask the user
OpenAI needs the to know what arguments you need to call your function. If you need some inputs from your user, you have many classic ways of getting it. If you are in a chatbot context , you can put in the prompt that at some point you'll need to get a data from the user and then let the conversation happen. At some point the LLM will ask the user the question et you'll need to parse it from the user response.

Just one last thing, while the function is being successfully executed I am unable to pass the response to GPT. Therefore the function is executed but there is not message sent to the user with the data that was given as a response by the function.

See this PR to have an example on how to pass the response to GPT.
$response = $this->chat->generateTextOrReturnFunctionCalled($prompt); => you have the response from the function, you can send it back to the user in a raw way. Or you can call GPT with the answer like in the PR and wait for GPT answer. Or you can call GPT with a custom prompt dedicated to what happen after your function is called.
Does this help you?

Hey,
I did help, thanks a lot.
I have another question. I'm trying to embed a document but it is returning an array whereas it should return a vector. How can I resolve this issue?

The array is the PHP representation of the vector. Where in your code does it create you a problem?

It creates an Issue when I'm saving it in elastic search. It returns Failed to parse object: expecting token of type [VALUE_NUMBER] but found [START_ARRAY]
When I log the response of
$embeddingGenerator = new OpenAIEmbeddingGenerator();
            $embedding = $embeddingGenerator->embedDocuments($formattedDocuments);
            \Log::info("embedding" . json_encode($embedding));

it returns "local.INFO: embedding[{"content":"[{"id":8719676342565,"title":"Gift Card...., embedding:[0.12232]".
the structure seems to be
[{
"content":[{
"embedding":[]
"sourceType":"files",
"sourceName":"/Users/mukulsharma/Documents/withoctober/storage/app/temp/places_1699270507.txt",
"hash":"b2306a679edeb05ad3e4aea1791ad12756a889dd0e8656844188d6e6a004a8ad",
"chunkNumber":0
}],
"content":[{
"embedding":[]
"sourceType":"files",
"sourceName":"/Users/mukulsharma/Documents/withoctober/storage/app/temp/places_1699270507.txt",
"hash":"b2306a679edeb05ad3e4aea1791ad12756a889dd0e8656844188d6e6a004a8ad",
"chunkNumber":0
}],
}]

Hello @absolutemadla8 ! Can you show me how you try to store the embedding the ES ?
You probably want to use the ElasticsearchVectorStore (see ElasticsearchVectorStoreTest.php).
If you want to store the embeddings without using our VectorStore, you should use "Dense vector" type in ES.

No news I'm closing this for now

repo: theodo-group/LLPhant


text:
I've always wanted this thanks!

Man I missed dial-up speeds.

We should patch this. This bug is a seriously vulnerability

authors: Kyokou cjhannah

repo: theonion/comcastifyjs


text:
[WEB][Image]

When we insert an image in a media card (in a story), it is supposed to be displayed then when the story is called.
But the current situation is that the bot doesn't display it at all we get an error 404.
The loophole would be to go in the "document section" where the image got uploaded, take it's link and insert the link in an <a href>, but that's not what we want, this solution is a bit too technical for métier teams.

It is not displayed in the bot ? With which connector ? Because it works for other bots :)

It's on the web connector, here is one example of what I get as output:

When I click on the link in the first picture, it does display correctly:

Maybe then this part wasn't made to handle pdf content, so I think it should be restricted to images as in .png, .jpeg and so on.
To avoid any confusion (if the previous sentence is correct)

authors: JordanSncf vsct-jburet

repo: theopenconversationkit/tock


text:
[Tock Studio] Refactor menus/tabs ?

Proposal for a refactoring of Tock Studio tabs and menus :

Several names (eg. NLU / QA) might be more intuitive for newcomers, in comparison with other solutions on the market
Several views/tabs might be moved to other menus : Logs from NLU, Bot Flow from Build, etc.

Please feel free to comment on this issue and propose changes.

Also see #932

Trying some renaming + icons + reordering...
What do you think @bakic @vsct-jburet @gsoumier @broxmik ?


text:
Fix: story disabled for single conf is disabled for all (when no other rule)

When a single rule exists for a story "disabled for config X", it is disabled for all configs.
To reproduce:

Create 2 configs : a and b
Create new story s (no rules yet)
Add single rule "story s disabled for config a"
Test the bot (story s is disabled for a and b) or simply refresh story-rules (it appears now for "All configurations")

Fixed with #742.


text:
Doc / mkdocs-material update

Hello @vsct-jburet,
Peux-tu stp me faire un retour sur ces 2 commits avant que je reprenne les pushs directs sur la documentation ?

je crois qu'il y a eu des copies du site commitées par erreur dans /docs-mk/*/site lors de la montée de version mkdocs-material
j'ai mis à jour mkdocs et mkdocs-material mais lorsque je regénère le site je vois quelques fichiers se faire supprimer ça me paraît louche, est-ce que tu penses que ma conf est ok ?
est-ce que c'est ok pour toi de proposer dans la doc ces configs cpu / ram / ec2 ?

Ma config : python 3.8 mkdocs 1.1.2 mkdocs-material 5.2.2

Merci.

Quand je lance le regen-docsite, j'ai cette erreur:

cp: illegal option -- u
usage: cp [-R [-H | -L | -P]] [-fi | -n] [-apvXc] source_file target_file
cp [-R [-H | -L | -P]] [-fi | -n] [-apvXc] source_file ... target_directory
-> j'ai fais la modif pour supprimer le -u sur la branche...

ok pour le fichiers corrigés par erreurs
je te push le diff sur la branche quand je fais un regen-docsite de manière à ce que tu vois le diff de mon côté - j'ai la même conf que toi sauf que je suis sur un python 3.7.3

OK on génère bien la même chose !

...hormis quelques erreurs de style codehighlight (Gradle n'existe pas il faut mettre Groovy de mon côté sinon j'ai un style error au rendu, contrairement à toi).

Il y a bien des fichiers svg / png / etc. supprimés du site mais a priori c'est pareil pour toi autrement tu les aurais re-générés j'imagine.

Pour le "DOCTYPE" j'ai corrigé la source

Et nettoyé -> 2 commits

authors: francoisno vsct-jburet


text:
Inconsistent Behavior

Environment:

Windows 10 x64
PHP 8.0.1
Package Version 3.3.4

So far in most cases where I'm using the Container, it seems to work pretty well. But I'm starting to get instances where the reflection just stops working altogether.
In many cases simply binding a concrete class to an abstract interface, or just sharing a concrete class on its own, just works. But it's behaving very inconsistently with some new structures I'm adding to my application.
For instance, I'm now having to explicitly set constructor arguments. In other places in my code, this does not have to be done.
$adapterDefinition = $container
	->share(ExceptionResponseAdapter::class)
	->addArguments([
		ResponseFactoryInterface::class,
		StreamFactoryInterface::class
	]);

I can retrieve and dump instances of those requirements before creating this definition and the container resolves them without issue, so I know it's not a matter of those dependencies failing to resolve. However, not explicitly setting the arguments here results in a 'too few arguments' error.
Likewise, it doesn't attempt to resolve dependencies with addMethodCall being given a class name, and instead passes said class name directly to the method, which obviously fails with a type error.
$adapterDefinition->addMethodCall('pipe', [$strategy]);

I have again confirmed that it's able to successfully resolve the $strategy independent of this, so I know the dependency isn't failing, and the container is simply refusing to follow instructions.

It's hard to see exactly what you're doing without seeing all of your definitions but ai suspect you're trying a combination of defined/reflection. In which case it works one or the other, if you define a service without it's arguments the container won't attempt to reflect.

I had the same problem. Here's the scenario:
interface EntityManagerInterface {}

class ConcreteEntityManager implements EntityManagerInterface {}

interface UserRepositoryInterface {}

class ConcreteUserRepository implements UserRepositoryInterface
{
    public function __construct(EntityManagerInterface $em) {}
}

class UserService
{
    public function __construct(UserRepositoryInterface $repository) {}
}

 // arrange
$container = new Container();
$container->delegate(new ReflectionContainer());

$container->add(EntityManagerInterface::class, ConcreteEntityManager::class);
$container->add(UserRepositoryInterface::class, ConcreteUserRepository::class);

// act
$entityManager   = $container->get(EntityManagerInterface::class);   // <<<----------- Ok!
$concreteUserRepository = $container->get(ConcreteUserRepository::class); // <<<----------- Ok!
$userRepository  = $container->get(UserRepositoryInterface::class); // <<<----------- Got Error!
$service = $container->get(UserService::class); // <<<----------- Got Error!

My questions:

If the container can resolve ConcreteUserRepository, why autowire cannot resolve an UserRepositoryInterface with an instance of ConcreteUserRepository?
Why extra configurations is required (addArguments()) if the container knows and is managing EntityManagerInterface?

Container setup:
$container = new League\Container\Container();

$container->delegate(
	new League\Container\ReflectionContainer()
);

Inside a Provider:
$this->leagueContainer->share(ExceptionResponseAdapterInterface::class, function () {
	die('never runs');
	return new ExceptionResponseAdapter([]);
});

$this->leagueContainer->share(ExceptionHandlerInterface::class, ExceptionHandler::class);

// dies: too few arguments, expects 1, gets 0
var_dump($this->leagueContainer->get(ExceptionHandlerInterface::class));
exit;

The ExceptionHandler class:
// AbstractExceptionHandler implements ExceptionHandlerInterface
class ExceptionHandler extends AbstractExceptionHandler
{
	public function __construct(protected ExceptionResponseAdapterInterface $exceptionResponseAdapter)
	{
		//
	}
}

I don’t see the service provider being added to the container?
Sent from ProtonMail for iOS
On Thu, May 6, 2021 at 17:55, Alan Ondra @.***> wrote:

Container setup:
$container = new League\Container\Container();
$container->delegate(
new League\Container\ReflectionContainer()
);
Inside a Provider:
$this->leagueContainer->share(ExceptionResponseAdapterInterface::class, function () {
die('never runs');
return new ExceptionResponseAdapter([]);
});
$this->leagueContainer->share(ExceptionHandlerInterface::class, ExceptionHandler::class);
// dies: too few arguments, expects 1, gets 0
var_dump($this->leagueContainer->get(ExceptionHandlerInterface::class));
exit;
The ExceptionHandler class:
// AbstractExceptionHandler implements ExceptionHandlerInterface
class ExceptionHandler extends AbstractExceptionHandler
{
public function __construct(protected ExceptionResponseAdapterInterface $exceptionResponseAdapter)
{
//
}
}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I'm just using $container->addServiceProvider(LoggingProvider::class);

My point is that I can’t debug it without seeing the whole thing, nothing missing
Sent from ProtonMail for iOS
On Thu, May 6, 2021 at 18:24, Alan Ondra @.***> wrote:

I'm just using $container->addServiceProvider(LoggingProvider::class);
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Okay, I'll just have to make a whole new project and attach it I guess? Right now aside from a bunch of code for the stuff I'm doing with my application that has nothing to do with this package, the related code is a barebones use of your package. I'll upload an example of it not working later.

Just a plain file:
<?php

require_once 'vendor/autoload.php';

interface IFoo
{
	//
}

class Foo implements IFoo
{
	//
}

interface IBar
{
	//
}

class Bar implements IBar
{
	public function __construct(protected IFoo $foo)
	{
		//
	}
}

try {
	$container = new League\Container\Container();
	$container->delegate(new League\Container\ReflectionContainer());

	$container->add(IFoo::class, Foo::class);
	$container->add(IBar::class, Bar::class);

	var_dump($container->get(IBar::class));
} catch (Throwable $exc) {
	printf(
		"%s # %d: %s (%s:%d)\n",
		get_class($exc),
		$exc->getCode(),
		$exc->getMessage(),
		$exc->getFile(),
		$exc->getLine()
	);
}

Running with php -f test.php:
ArgumentCountError # 0: Too few arguments to function Bar::__construct(), 0 passed and exactly 1 expected (/home/alan/Workspace/Container Test/test.php:22)

You haven’t added IFoo as an argument for IBar
Sent from ProtonMail for iOS
On Thu, May 6, 2021 at 19:17, Alan Ondra @.***> wrote:

Just a plain file:
delegate(new League\Container\ReflectionContainer());

	$container->add(IFoo::class, Foo::class);
	$container->add(IBar::class, Bar::class);

	var_dump($container->get(IBar::class));
} catch (Throwable $exc) {
	printf(
		"%s # %d: %s (%s:%d)\n",
		get_class($exc),
		$exc->getCode(),
		$exc->getMessage(),
		$exc->getFile(),
		$exc->getLine()
	);
}

Running with php -f test.php:

ArgumentCountError # 0: Too few arguments to function Bar::__construct(), 0 passed and exactly 1 expected (/home/alan/Workspace/Container Test/test.php:22)

—
You are receiving this because you commented.
Reply to this email directly, [view it on GitHub](https://github.com/thephpleague/container/issues/222#issuecomment-833751436), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AAP3E2OILZJSWETJHPY22H3TMLMKVANCNFSM4YDALT5Q).

Yes. I shouldn't have to.

Erm, yes you should, it doesn’t mix reflection and definitions, and won’t ever going forward.
Sent from ProtonMail for iOS
On Thu, May 6, 2021 at 19:29, Alan Ondra @.***> wrote:

Yes. I shouldn't have to.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Noted.

authors: alanondra brenoroosevelt philipobenito

repo: thephpleague/container


text:
Fix: Inject delegating container into delegated container

This PR

[x] asserts that the container to which is delegated can resolve items from the delegating container
[x] injects delegating container into delegated container

Fixes https://github.com/thephpleague/container/issues/78.

Build failing (not checked) but assuming because of ::class constants in the test

Thank you, @philipobenito!

:+1: will get merged and tagged shortly

Thanking you :+1:

You're welcome, @philipobenito!

authors: localheinz philipobenito


text:
Build suddenly failing

Hi gang,
I installed last night with "league/csv": "~7.0" in my composer.json, and today my builds are failing:

Installing league/csv (dev-master 0d3c28b)
Cloning 0d3c28bf20ad26e1e23599e1746aaf7c680c0477
0d3c28bf20ad26e1e23599e1746aaf7c680c0477 is gone (history was rewritten?)
Failed to download league/csv from source: Failed to execute git checkout '0d3c28bf20ad26e1e23599e1746aaf7c680c0477' && git reset --hard '0d3c28bf20ad26e1e23599e1746aaf7c680c0477'
fatal: reference is not a tree: 0d3c28bf20ad26e1e23599e1746aaf7c680c0477
Now trying to download from dist

Installing league/csv (dev-master 0d3c28b)
Downloading: connection...

[Composer\Downloader\TransportException]
Could not authenticate against github.com

Did I screw something up?

(I needed 7.0 because it was not parsing my CSV line breaks correctly without $csv->setNewline("\r\n"); )

Hi,
thanks for using the League\Csv. The branch you are currently using is the dev-master branch. So this can happen when some features are not enough stable and I have to make some git stash/rebase.
I would recommend using the dev-master branch only for testing purpose and not for production code.
The code will be freeze today for release hopefully next week.

the stable release of 7.0.0 is out !!

authors: nyamsprod snipe

repo: thephpleague/csv


text:
[Feature Request] Empty Includes

I'd like to be able to get an empty set of embedded data.
{
"id" : 4,
"address" :
{
"street" : "123 Main",
"city" : "Chicago",
"state" : "IL",
"zip" : "12354",
}
}
This will allow for consistent data structures.
So a similar record without an address will look like:
{
"id" : 4,
"address" :
{
}
}

I think this is more a case for a custom serializer than core functionality.

authors: texmorgan willishq

repo: thephpleague/fractal


text:
replacement content

Hi!
Just wondering if could be possible to have a public method to replace current content of modal by other one, but keeping opened the same modal itself.
I've been checking the library and I found _releaseNode  method, that does exactly what I need, although calling it directly the current property is not updated. Could  be created a public property that refers this method and update whatever is need around this change?

Hi @smoruetasantiago,
You can do this already by using modal.open. Try it out on the demo page:

Open the developer console.

Type modal.open('#modal-1'), hit .

Type modal.open('#modal-2'), hit .

You'll see that using modal.open will release the current DOM node and replace the content in the modal with that of the element provided to it as an argument.

I'm closing this issue for now @smoruetasantiago. I hope the solution above worked for you. Please let me know if you need any further support.

authors: benceg smoruetasantiago

repo: thephuse/vanilla-modal


text:
Refactor GameManager to use new Repository structure

Like the UserManager, the GameManager needs to be changed to use the DbRepository instead of accessing  the database directly.
Use UserManager as an example

@m-soup I think you're working on this. If that is the case, can you assign this to yourself and move it over in the Projects section of github?

Closing implemented issue.

authors: m-soup theprimegamer


text:
Update cli.ts to work on more machines when called like ./cli.ts --args here

Description
It seems that shebang by default treats everything after a space as a single argument, so npx + ts-node is treated like looking for npx ts-node executable. env itself suggested me adding -S flag to fix it:

dzek@dzek-Mint:/Projects/tmp/privacy-protect$ LC_ALL=C ./cli.ts encrypt -m "My secret" --hint "My hint" --deniableMessage "My deniable secret" ./out/secret.html
/usr/bin/env: 'npx ts-node': No such file or directory
/usr/bin/env: use -[v]S to pass options in shebang lines

This should make it work on more machines if it even works on some right now - I'm not sure about this. However on Ubuntu 22.04 this is required to make it work. The -S flag was probably added around 2017. I suspect if that's a conflict we should prefer newer systems.
Test plan

Clone repo
Run npm install
Run ./cli.ts encrypt -m "My secret" --hint "My hint" ./secret.html

Possible regressions or performance implications
If -S flag is unknown to some old Linux distributions it may introduce a regression. I have no idea about version usage of Unix-like operating systems, but I believe everything that doesn't support this flag is pretty low on usage and we should prefer newest ones anyway.
Dependencies
none
Reviewer checklist

[X] Code is well-designed and only as complicated as necessary.
[X] Good experience for clients of code.
[X] Existing use-cases are implemented, not potential future ones (YAGNI).
[X] Clear and consistent naming.
[X] Comments and documentation are clear, useful, and explain why instead of what.
[X] Sensible and attractive UI changes, if any.

Looks great, thanks for the detailed description of the change! It was working without the -S on macOS Ventura 13.1 and continues to work with it.

repo: therockstorm/privacy-protect


text:
Duplicated folder item found

First of all, thank you very much for this extension!
I'm trying to use it with an existing project that already contains Tailwind but Visual Studio cannot load my solution project (load failed).
When I try to rebuild it, I always get the error: 'Duplicated folder item found in the project: "wwwroot\css\tailwind-input.css"'.
I deleted this file to test but I still get the same error.
I also deleted the tailwind.config.js file to test.
Is there anything special to do on an existing project?

Found how to solve it.
Had to open .csproj and comment :
  <ItemGroup>
    <Folder Include="wwwroot/css/tailwind-input.css" Set="Css" />
    <Folder Include="tailwind.config.js" Set="Css" />
  </ItemGroup>

Then build and uncomment it.


text:
feat: Add Bonus Synchronicity Points to HSR notes

Add Bonus Synchronicity Points to HSR notes

Names should be aliased to something more intuitive.

looks good to me

authors: kgirtxd omg-xtao seriaati


text:
🛑 4id is down

In 41f3114, 4id (https://fourid.theswissbay.ch/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: 4id is back up in 4b597ec.

repo: theswissbay/status


text:
Changes to the Recognition of Regions

As per Board of Directors request via email. The WST is not making any political statements about China or Chinese diplomacy. We are acting on a binding instruction of our supervising Board authority.
Please direct all relevant discussions and opinions towards the public or internal mail threads. Discussion comments under this PR that go beyond the technical implementation will be disregarded.

As of writing this comment, the "Upcoming" tab on the homepage at https://live.worldcubeassociation.org/ does not show the intended flag from this PR next to "New Taipei Open 2023"

@gregorbg it shows up as expected for me. Can you try closing the WCA Live tabs and opening again or trying in incognito? It's likely a caching issue.

That was a caching issue indeed :facepalm:
But I am still surprised to see that it was a caching issue across reboots on my machine, is this something that you are controlling from the application side? Or is this my browser running haywire?

This happened a couple times and one theory I have is that on some devices the old service worker hasn't been deactivated yet (and it would do more aggressive caching). We used to have a service worker but it only caused troubles so I disabled it like 10 months ago. The confusing part is that it should definitely be deactivated if the user visited WCA Live several times since that change. Currently I don't have a better idea, we don't do any unusual caching and all assets have hash in their name.

Hm, in that case I guess we're as fine as we can reasonably be. Thanks!

authors: gregorbg jonatanklosko

repo: thewca/wca-live


text:
Timeline: Make it possible to have a different response than checkmarks

From Norway:
The response here when clicking "Check answers" is the same. It would be useful with two responses here: All is correct – and "some are correct, you have got 3/5 correct". Of course, the text should be editable.

This is the timeline page (normal drag and drop doesn't let you drop on wrong answer)

authors: FayCross torinfo

repo: thexerteproject/xerteonlinetoolkits


text:
pushAndRemoveUntil?

Hey there,
support pushAndRemoveUntil?
Thanks!

If you set up onGenerateRoute: myrouter.generator, on your main application class then you can use the standard Navigator.pushAndRemoveUntil method. I am not planning to wrap every function on navigator, but I could add a predicate in a future version.

authors: iniwf lukef

repo: theyakka/fluro