Skip to content

get rid of Notice 'Only variable references should be returned by reference'#135

Merged
chdemko merged 1 commit intojoomla:masterfrom
piotr-cz:patch-2
Feb 24, 2012
Merged

get rid of Notice 'Only variable references should be returned by reference'#135
chdemko merged 1 commit intojoomla:masterfrom
piotr-cz:patch-2

Conversation

@piotr-cz
Copy link
Contributor

php 5+, tested on php 5.3.0

@infograf768
Copy link
Member

Please create a tracker on joomlacode.

@piotr-cz
Copy link
Contributor Author

@infograf768 is it really required for such small thing?

@infograf768
Copy link
Member

Yes.
The tracker is the way to go for every patch, small or big.
Thanks for your understanding.

@piotr-cz
Copy link
Contributor Author

No problem, added issue yesterday, just it didn't show yet.

chdemko added a commit that referenced this pull request Feb 24, 2012
get rid of Notice 'Only variable references should be returned by reference'
@chdemko chdemko merged commit 1f4224a into joomla:master Feb 24, 2012
bembelimen referenced this pull request in bembelimen/joomla-cms Apr 26, 2018
wilsonge pushed a commit to wilsonge/joomla-cms that referenced this pull request May 26, 2019
Use JHIDE and JSHOW for Options in mod_quickicons
richard67 pushed a commit to richard67/joomla-cms that referenced this pull request Apr 3, 2020
Fedik pushed a commit to Fedik/joomla-cms that referenced this pull request Dec 17, 2021
bembelimen added a commit that referenced this pull request Dec 29, 2021
* jooa11y scaffolding

media folder
web assets
     hard coded until we decide on npm or not

to install check out this branch
npm run build:js
npm run build:css
discover install the plugin

todo - aka brian needs help
[] make the checker start
[] make the checker use the settings from the plugin

* script loads now

* tippy - temp

* editor button

* toolbar

* enpugh

* Jooa11y/enhancements (#132)

* Add some enhancements

* Fix wrong name

Co-authored-by: Benjamin Trenkle <benjamin.trenkle@wicked-software.de>

* tabs

* tippy popper licence and naming

* postgresql install

* xml bug

* cleaning

* tidy

* fix

* dont need scss

* needs to be an integer

* deploy version

* might not be the best way?

* Revert "Auxiliary commit to revert individual files from 18199dd"

This reverts commit 4eb15800ccc4bb0d64580013b58a92505bc5abfc.

* clarify show

* options

* package.lock

* remove editor button

* cs

* string

* set input

* deploy

* Temporary add git to node ci (#133)

* github link

* param

* Revert "param"

This reverts commit 378a64e.

* params

* spelling

* revert

* sql field

* cs and spelling

* Fix npm

* Revert drone.yml change

* Switch package to tarball

* Use the es module

* Update eslint-plugin-vue to 8.2.0

* CS (#135)

* lang

* progress update

* fix strings

* load strings

* js change from @Fedik

* typo and package-lock

* sef and non-sef

* revert change by Dimitris

* Update joomla-a11y-checker version

* Update joomla-a11y-checker version

* Update view preview url

* simplify

* Update jooa11y.php

* fix jscs (#136)

* undo eslint change

* remove "Additional ruleset toggles" from the plugin settings

* Update .eslintrc

lets try this again

* final try

* sha

Co-authored-by: Benjamin Trenkle <benjamin.trenkle@wicked-software.de>
Co-authored-by: Harald Leithner <leithner@itronic.at>
Co-authored-by: dgrammatiko <d.grammatiko@gmail.com>
Co-authored-by: Dimitris Grammatikogiannis <dg@dgrammatiko.dev>
Co-authored-by: Benjamin Trenkle <bembelimen@users.noreply.github.com>
Co-authored-by: Fedir Zinchuk <getthesite@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants