WiFi: saves a file for each text change.

Bug #1761713 reported by Alexander Traud
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
QR Tools
Fix Released
Undecided
Ramiro Algozino

Bug Description

Facing the same issue with the latest trunk (currently revision 29), which was originally reported at <https://bugs.launchpad.net/ubuntu/+source/qr-tools/+bug/1729066>: ‘QtQR writes a file every time I introduce a character of any of the fields of the WiFi network (SSID, PASSWORD, encryption) [in t]he directory where I launch QtQr from[.]’

Steps to Reproduce:
1a. Ubuntu 17.10 with the package qr-tools installed
1b. Download the latest QtQR and qr-tools from
    <https://bazaar.launchpad.net/~qr-tools-developers/qr-tools/trunk/files>
2. Open a Terminal
3. $ cd ~/Downloads
4. $ ./qtqr
5. Select data type: WiFi Network
6. Enter a SSID

Expected Result:
Although QtQR creates a temporary representation, it should not be visible as file.

Actual Result:
QtQR writes a file from where QtQR was launched, for each text change. When I type EXAMPLE, I end up with seven PNG files (E.png, EX.png, ..., EXAMPLE.png) in the current working directory.

Notes:
The traceback is
self.wifi...textChanged.connect
-> self.qrencode (qtqr)
--> qr.encode (qrtools)
---> filename or get_tmp_file
The filename is set (as Unicode) and therefore not a temporary but an actual file is created. The same happens for every QLineEdit. Consequently, this happens on the SMS type when I enter a telephone number as well. It does not happen with QPlainTextEdit because of a different textChanged behavior. Not sure how to tackle this or when this issue was introduced (with the change from Qt4 to Qt5 or even older).

Related branches

Revision history for this message
Ramiro Algozino (algozino) wrote :

So, when connecting the signal and triggering it, the signal passes along paremters for QLineEdits with the QLineEdit content; the qrencode method receives this parameter as "fileName" and then generates one file for each time the QLine is triggered (the same applies to other cases).

We'll fix this tomorrow.

Thanks!

Changed in qr-tools:
status: New → Confirmed
assignee: nobody → Ramiro Algozino (algozino)
Changed in qr-tools:
status: Confirmed → Fix Committed
Changed in qr-tools:
status: Fix Committed → Fix Released
Revision history for this message
Alexander Traud (traud) wrote :

I have to say thanks! Bug is fixed.

Do I have to do anything else or report somewhere else? If I understand correctly, somebody in Debian packages your project. Then, that package drops back into the world of Canonical (Ubuntu, …). Shall I report this with the packages as well, so their maintainers pull your fixes? Both, Debian and Ubuntu are still on revision 23.

Revision history for this message
Ramiro Algozino (algozino) wrote : Re: [Bug 1761713] Re: WiFi: saves a file for each text change.

Hi Alex,

I don't know who's mantaining the Debian packages, if you can report it and
make them update it, that woudl be great!

Thanks!

El vie., 15 de jun. de 2018 a la(s) 06:32, Alexander Traud (
<email address hidden>) escribió:

> I have to say thanks! Bug is fixed.
>
> Do I have to do anything else or report somewhere else? If I understand
> correctly, somebody in Debian packages your project. Then, that package
> drops back into the world of Canonical (Ubuntu, …). Shall I report this
> with the packages as well, so their maintainers pull your fixes? Both,
> Debian and Ubuntu are still on revision 23.
>
> --
> You received this bug notification because you are a member of QR Tools
> Team, which is subscribed to QR Tools.
> https://bugs.launchpad.net/bugs/1761713
>
> Title:
> WiFi: saves a file for each text change.
>
> Status in QR Tools:
> Fix Released
>
> Bug description:
> Facing the same issue with the latest trunk (currently revision 29),
> which was originally reported at
> <https://bugs.launchpad.net/ubuntu/+source/qr-tools/+bug/1729066>:
> ‘QtQR writes a file every time I introduce a character of any of the
> fields of the WiFi network (SSID, PASSWORD, encryption) [in t]he
> directory where I launch QtQr from[.]’
>
> Steps to Reproduce:
> 1a. Ubuntu 17.10 with the package qr-tools installed
> 1b. Download the latest QtQR and qr-tools from
> <
> https://bazaar.launchpad.net/~qr-tools-developers/qr-tools/trunk/files>
> 2. Open a Terminal
> 3. $ cd ~/Downloads
> 4. $ ./qtqr
> 5. Select data type: WiFi Network
> 6. Enter a SSID
>
> Expected Result:
> Although QtQR creates a temporary representation, it should not be
> visible as file.
>
> Actual Result:
> QtQR writes a file from where QtQR was launched, for each text change.
> When I type EXAMPLE, I end up with seven PNG files (E.png, EX.png, ...,
> EXAMPLE.png) in the current working directory.
>
> Notes:
> The traceback is
> self.wifi...textChanged.connect
> -> self.qrencode (qtqr)
> --> qr.encode (qrtools)
> ---> filename or get_tmp_file
> The filename is set (as Unicode) and therefore not a temporary but an
> actual file is created. The same happens for every QLineEdit. Consequently,
> this happens on the SMS type when I enter a telephone number as well. It
> does not happen with QPlainTextEdit because of a different textChanged
> behavior. Not sure how to tackle this or when this issue was introduced
> (with the change from Qt4 to Qt5 or even older).
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/qr-tools/+bug/1761713/+subscriptions
>

--
Ramiro Algozino
http://algozino.com.ar/

Revision history for this message
Mark (mark-harrison23) wrote :

This bug still exists (2020 September 6)

Revision history for this message
Ramiro Algozino (algozino) wrote :

Hi Mark,

Could please check that you are using the latest version from the Daily PPA?

Thanks.

On Sun, Sep 6, 2020, 16:41 Mark <email address hidden> wrote:

> This bug still exists (2020 September 6)
>
> --
> You received this bug notification because you are a member of QR Tools
> Team, which is subscribed to QR Tools.
> https://bugs.launchpad.net/bugs/1761713
>
> Title:
> WiFi: saves a file for each text change.
>
> Status in QR Tools:
> Fix Released
>
> Bug description:
> Facing the same issue with the latest trunk (currently revision 29),
> which was originally reported at
> <https://bugs.launchpad.net/ubuntu/+source/qr-tools/+bug/1729066>:
> ‘QtQR writes a file every time I introduce a character of any of the
> fields of the WiFi network (SSID, PASSWORD, encryption) [in t]he
> directory where I launch QtQr from[.]’
>
> Steps to Reproduce:
> 1a. Ubuntu 17.10 with the package qr-tools installed
> 1b. Download the latest QtQR and qr-tools from
> <
> https://bazaar.launchpad.net/~qr-tools-developers/qr-tools/trunk/files>
> 2. Open a Terminal
> 3. $ cd ~/Downloads
> 4. $ ./qtqr
> 5. Select data type: WiFi Network
> 6. Enter a SSID
>
> Expected Result:
> Although QtQR creates a temporary representation, it should not be
> visible as file.
>
> Actual Result:
> QtQR writes a file from where QtQR was launched, for each text change.
> When I type EXAMPLE, I end up with seven PNG files (E.png, EX.png, ...,
> EXAMPLE.png) in the current working directory.
>
> Notes:
> The traceback is
> self.wifi...textChanged.connect
> -> self.qrencode (qtqr)
> --> qr.encode (qrtools)
> ---> filename or get_tmp_file
> The filename is set (as Unicode) and therefore not a temporary but an
> actual file is created. The same happens for every QLineEdit. Consequently,
> this happens on the SMS type when I enter a telephone number as well. It
> does not happen with QPlainTextEdit because of a different textChanged
> behavior. Not sure how to tackle this or when this issue was introduced
> (with the change from Qt4 to Qt5 or even older).
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/qr-tools/+bug/1761713/+subscriptions
>

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.